build: consolidate merged WP0 stack onto main - #40
Merged
Conversation
Apply the net #28 through #32 contribution, governance, workflow hardening, Windows checkout, and strict Make changes to main after #35, #36, and #38, without importing the stacked commit history. Constraint: Preserve main's check-portable, dependency-license evidence, and dependency override while excluding the duplicate #34 portable gate and superseded #37 license implementation. Rejected: Merging the final stacked branch wholesale, retaining duplicate portable or license jobs, or deleting historical branches before the consolidated result is published. Confidence: Conflict resolutions retain the newer main implementations, unchanged endpoint blobs where appropriate, and a single portable and license path; fresh-checkout RED evidence identified missing module lock LF attributes. Scope-risk: The commit changes governance and CI contracts but does not change public API, runtime behavior, persistence, or generated transport contracts. Tested: In a fresh Windows checkout, go mod tidy -diff; go mod verify; governance unit tests and gate; focused release, Make, and workflow tests; targeted vet; pinned lint with zero issues; actionlint; check-generated; four-platform check-portable; Windows LF and frozen-hash contract; git diff --check. Not-tested: Full-repository vet, lint, and dependency-license binary regeneration on this final commit. Native Windows stops at the existing SeekDB Linux/macOS-only CGO loader, and the local-only branch has no exact-Head Linux CI. Co-authored-by: OmX <omx@oh-my-codex.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tracking
Part of #3Problem and rationale
PRs #28 through #32 were merged through an intermediate stacked branch chain, so their contribution, governance, workflow-hardening, Windows checkout, and strict Make contracts did not reach
main. Merging the final stacked Head wholesale would also reintroduce portable and dependency-license implementations already superseded by #35 and #36. This PR applies only the missing net behavior onto the currentmain.Changes
check-portable, ci: verify dependency license evidence #36 dependency-license evidence, and fix(deps): override vulnerable ansi-regex #38 dependency override.go.modandgo.sumafter a fresh Windows checkout reproduced whole-file module diffs.Behavior and compatibility
Generated, dependency, and workflow impact
go.modandgo.sumare complete and verified.Validation
Commands run against submitted Head
1a992ef8bbe0aed8205c74e2b71de92892d869aein a fresh Windows checkout:All commands above passed; pinned lint reported
0 issues, and all four portable targets built successfully.Native Windows cannot complete full-repository
make check,make lint, ormake license-dependenciesbecause the existing SeekDB implementation includesloader.cwhile its CGO Go implementation is build-tagged for Linux/macOS. Exact-Head Linux CI is therefore required before merge.git diff --checkgit status --shortwas inspected after validation.AI usage
AI assistance was used to reconstruct the stacked PR topology, resolve the integration conflicts, add the LF regression coverage, and prepare the consolidation commit. The resulting Head was independently checked through fresh-checkout module, governance, workflow, generated-contract, portable-build, lint, and raw-byte verification described above.