[codex] Add artifact identity to center entry map#247
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ad9dff8a15
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 519a868578
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "surface_state": { | ||
| "type": "string", | ||
| "minLength": 3 | ||
| }, |
There was a problem hiding this comment.
Constrain surface_state to the canonical vocabulary
The generated payload now uses generated, but this new schema entry still accepts any arbitrary string for artifact_identity.surface_state, including the previously rejected composite value. Since docs/organ-contract/SURFACE_STATES.md defines the canonical state vocabulary and this schema is the machine-facing contract for generated/center_entry_map.min.json, consumers validating only the schema would still accept non-classifiable surface states; make this an enum of the canonical states, or const: "generated" for this readmodel.
Useful? React with 👍 / 👎.
Adds artifact identity posture to the Agents-of-Abyss center entry map and route contract.\n\nValidation: python scripts/release_gate/release_check.py.