Skip to content

chore(review): remove overengineered validation guards#21

Merged
ssfun merged 1 commit into
mainfrom
codex/remove-review-overdesign
Jul 22, 2026
Merged

chore(review): remove overengineered validation guards#21
ssfun merged 1 commit into
mainfrom
codex/remove-review-overdesign

Conversation

@ssfun

@ssfun ssfun commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove redundant daily full latest-upstream CI while retaining manual latest validation and automated release gates
  • simplify overlay provenance to reviewed upstream hashes while preserving fail-closed collision checks
  • retire implementation-shape and brittle workflow/Docker string-scanning tests
  • record the simplification rationale and retained risk controls in the review report

Validation

  • repository validation: 19 Management customization/security tests and 2 reproducible archive tests
  • fresh Management v1.18.5: repeated application, 110 Bun tests, ESLint, TypeScript, production build
  • fresh Core v7.2.94: guarded preflight, rejected reapplication, target Go packages, server build
  • source Docker build deferred to the required source-image-build PR check because Dockerfiles are unchanged

@ssfun
ssfun merged commit 0dbbc94 into main Jul 22, 2026
5 checks passed
@ssfun
ssfun deleted the codex/remove-review-overdesign branch July 22, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant