SLOT-FCP-CLEANUP (factory): remove the temporary sequencing shims - #7
Merged
Merged
Conversation
Delete schemas/index.ts and template_registry.ts plus their force-added dist compilations and package.json export/files entries. The shims existed only so afi-reactor main compiled before SLOT-FCP-REACTOR; the reactor's dependency cut has merged, so nothing imports them. Drop the shim notes from README, AGENTS, and the codex; record afi-reactor as a consumer of the authored artifacts. Authorized by afi-governance decisions/factory-configurable-pipelines-v1.md (D-FCP-9), gates passed: reactor 32001da, seeding f91ce44, factory 9f88ede, infra d263154. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…d hashing spec) Re-vendor the closure at afi-config origin/main f91ce4465b9c54bc221ba82e7a468544ffcf3fe3. Only canonical-json-hashing.v1.md changed: the W3a amendment (per accepted D-FCP-7) makes the spec's exclusion table carry the registered afi.d2.composition-manifest / afi.d2.analyst-config / afi.d2.plugin-set tags and the derived afi.plugin-set.v1 hash-material rule the shipped implementation already follows. All eight schemas and the KAT file are byte-identical at the new pin; digest algorithm and KAT vectors are unchanged, so no official hash moves. MANIFEST sha256 + commit, the CI pinned-checkout ref, and every doc/test pin reference updated together. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The W3a amendment landed in the re-pinned vendored spec: its exclusion table now lists the registered afi.d2.* composition tags and the derived afi.plugin-set.v1 rule the implementation ships. Drop the README's spec-table divergence caveat and cite the spec section directly. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
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
1 participant
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.
Slot: SLOT-FCP-CLEANUP / final SLOT-FCP-FACTORY pass (D-FCP-9). The afi-reactor dependency on afi-factory is cut (merged) and the gateway boundary-proof fixture no longer builds an afi-factory-importing reactor, so nothing imports the shims.
Deletes the two
TEMPORARY SEQUENCING SHIMfiles (schemas/index.ts,template_registry.ts) + their dist compilations + package.json export entries; re-pins the vendored governed-schema closure to afi-configf91ce44(only the amended hashing spec changed — no official artifact hash moved); removes the README spec-table divergence note now that the amended spec matches the shipped D-FCP-7 tags/rule.127/127 tests green from a clean clone; zero shim references remain.
🤖 Generated with Claude Code