-
Notifications
You must be signed in to change notification settings - Fork 1
agents: Edit 1 — external-conversation absorbs land research-grade, not operational (Amara 5th-ferry) #248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -198,6 +198,29 @@ These apply to any AI harness. | |||||||||||||||||||||||||||||||||
| workflow at `docs/DECISIONS/YYYY-MM-DD-*.md` | ||||||||||||||||||||||||||||||||||
| rather than burying the rationale in a commit | ||||||||||||||||||||||||||||||||||
| message. | ||||||||||||||||||||||||||||||||||
| - When an agent ingests an external conversation — | ||||||||||||||||||||||||||||||||||
| courier ferry, cross-AI review, ChatGPT paste, | ||||||||||||||||||||||||||||||||||
| other-harness transcript — the absorb lands | ||||||||||||||||||||||||||||||||||
| research-grade, not operational. Concretely: | ||||||||||||||||||||||||||||||||||
| the absorb doc carries `GOVERNANCE.md §33` | ||||||||||||||||||||||||||||||||||
| archive headers including | ||||||||||||||||||||||||||||||||||
|
Comment on lines
+205
to
+206
|
||||||||||||||||||||||||||||||||||
| the absorb doc carries `GOVERNANCE.md §33` | |
| archive headers including | |
| the absorb doc carries archive headers including |
Copilot
AI
Apr 24, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P1: docs/DRIFT-TAXONOMY.md does not exist in the current tree (only references to it). If the worked example is meant to point at a promoted operational doc, the path needs to be updated to the actual file, or the example should reference an artifact that exists on this branch.
| absorb itself. Worked example: the drift-taxonomy | |
| promotion from | |
| `docs/research/drift-taxonomy-bootstrap-precursor-2026-04-22.md` | |
| (research-grade absorb) to | |
| `docs/DRIFT-TAXONOMY.md` (operational one-page | |
| field guide) — the absorb stayed in-place as | |
| provenance; the promotion is the ratification. | |
| absorb itself. Worked example: a drift-taxonomy | |
| absorb such as | |
| `docs/research/drift-taxonomy-bootstrap-precursor-2026-04-22.md` | |
| stays in place as provenance, while the | |
| ratified current-state promotion lands in an | |
| operational artifact that exists in the live doc | |
| set, such as an ADR under `docs/DECISIONS/` or | |
| a BP-NN entry in | |
| `docs/AGENT-BEST-PRACTICES.md`. |
Copilot
AI
Apr 24, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR description says this edit avoids introducing parallel “staged/ratified” terminology, but this line introduces “ratification” anyway. Consider keeping the vocabulary consistent (e.g., stick to “promotion” / “operational”) to avoid reintroducing the term you’re trying to avoid.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This new rule makes
GOVERNANCE.md §33the authority for required absorb headers, but in this commitGOVERNANCE.mdonly has numbered rules through §32. Contributors following this instruction cannot find the schema, so the policy is not actually enforceable and different agents can emit incompatible archive metadata. Please either add §33 in the same change or point this bullet at an existing section.Useful? React with 👍 / 👎.