- Rewrote the remaining em dashes outside docs/reviews/ (preserved as historical record, matching the alktty precedent): tests/ (313), fuzz/ (34 plus one shell comment), tasks/ (671 plus 12 kept in fences), docs/research + sdd_process + plans (525 plus 25 in fences), docs/architecture/overview.md fence labels (13), README, CHANGELOG, AGENTS.md, fuzz/README.md (176), and the .opencode/agents prompts (110) - Rewrote fenced content too: ASCII tree and module-map label separators, Rust snippet comments, and template placeholders; wire data, JSON/BAST examples, and transcripts survive verbatim - Rewrote 6 assert/expect message literals in tests/ and 2 format strings in fuzz/shared/src; no test data changed - Added Project Convention 18: no em dashes in prose (doc comments, markdown including docs/ and tasks/, agent prompts, commit messages); fence content rule, en-dash clause rule, and the no-spaced-hyphen prohibition follow alktty's convention 15 Follows6b20924(src doc comments) andcd0afe0(docs/architecture). Verification: cargo test (141 pass); cargo clippy --all-targets -D warnings; cargo fmt --check; cargo doc --no-deps; wasm check and clippy for wasm32-unknown-unknown with -D warnings; em dash grep over the repo returns matches only in docs/reviews/
5.0 KiB
5.0 KiB
id, name, status, depends_on, scope, risk, impact, level, tags
| id | name | status | depends_on | scope | risk | impact | level | tags | |||
|---|---|---|---|---|---|---|---|---|---|---|---|
| architecture/oq-promotion-sync | Sync AGENTS.md + phase-0 ledger pointers to the promoted OQ ledger (planning) | completed | single | trivial | project | planning |
|
Description
Phase 1 promoted the Phase 0 OQ ledger into docs/architecture/open-questions.md
(OQ-TN-01..14) and split OQ-TN-11/13 into resolved halves (ADR-001/ADR-003).
The older ledgers still describe pre-Phase-1 states in two places:
docs/research/phase-0-findings.md: the OQ sections (OQ-TN-01..10) should carry one-line "promoted todocs/architecture/open-questions.mdas OQ-TN-NN" pointers so a reader landing in the research doc finds the live tracker (the convergence checklist's last unchecked item says "promoted ... with statuses"; the promotion exists, and the back-pointer is the missing half).AGENTS.md: the "Architecture Context" section saysdocs/architecture/"will hold the authoritative spec (Phase 1 of the SDD process)... It does not exist yet". That is now stale: the spec exists (ADR-001..006 + 4 spec docs + BAST + OQ tracker). Update the paragraph to point at the real directory and adjust the "read it before non-trivial changes once it exists" line. Three adjacent stale references in the same file fold in (same staleness class, one pass; found during task review 2026-09-08):- Convention 16 (module structure): "expected shape (to be finalized in the architecture docs)"; the docs now exist and pin the module shape; state it as decided (producer half / consumer half / shared wire).
- Convention 11: "alkcall is v0.5.x". Cargo.toml pins
0.7.0, and the convention's own later text describes the 0.6.0/0.7.0 surfaces; the version sentence is self-contradicting. - Architecture Context alkcall bullet: "alkcall (v0.4.x, crates.io)", the same version drift.
Work
- phase-0-findings.md: add a promotion note at the top of the Open Questions
section (and one line under the convergence checklist's final item) pointing
at
docs/architecture/open-questions.md; mark the checklist item done. - AGENTS.md: rewrite the
docs/architecture/bullet to the present tense (spec exists; read it before non-trivial changes; ADRs 001..006 are Accepted; the OQ tracker is authoritative for open questions). Also fix the three folded stale references: convention 16's "to be finalized" hedge, and the two alkcall version mentions (v0.5.x in convention 11, v0.4.x in the Architecture Context bullet → 0.7.0).
Verification
- phase-0-findings.md has the promotion pointer; the checklist's final item is checked
- AGENTS.md architecture-context paragraph reflects the existing spec
- Convention 16 states the module shape as decided; no "to be finalized" hedge remains
- No alkcall version drift in AGENTS.md (convention 11 + Architecture Context bullet both say 0.7.0, matching Cargo.toml)
- No other stale "does not exist yet" language anywhere in AGENTS.md or docs/
Out of scope
- Any change to the OQ statuses themselves (that is the tracker's job now)
References
- docs/architecture/open-questions.md (the promoted ledger)
- docs/sdd_process.md §Safe Exit (the two-halves rule for deferrals)
Notes
- Executed together with
architecture/oq-tn-12-trackercreation (process-gap fix found during task review; OQ-TN-12's Blocked-on text now references its tracker task id persdd_process.md§Deferred OQs and the blocker-task half). - While rewriting the Architecture Context bullet, a fourth same-class
stale line surfaced and folded in: "The remaining Phase 1 work is the
spec" → Phase 1 complete, Phase 2 decomposed in
tasks/. - phase-0-findings.md: promotion note added at the top of the Open
Questions section (pointer to
docs/architecture/open-questions.md, Phase 0 statuses marked as non-maintained record); the convergence checklist's final item checked with the promotion details. - AGENTS.md: architecture bullet rewritten to present tense; convention 16 de-hedged (shape pinned by ADR-004/ADR-005); convention 11 v0.5.x → v0.7.x; Architecture Context alkcall bullet v0.4.x → v0.7.x; Phase 1 work bullet updated to complete.
- Grep sweep for "does not exist / not exist yet / will hold the / once it exists / to be finalized / v0.4.x / v0.5.x / remaining Phase 1 work is the spec" over AGENTS.md + docs/ is clean (only the task's own description and phase-0-findings.md's unrelated "Sentinel collision does not exist" line match).
Summary
Phase-0 OQ ledger back-pointers landed
(docs/research/phase-0-findings.md promotion note + checked-out
convergence checklist) and AGENTS.md is current with the completed
Phase 1 spec (architecture bullet, convention 16, convention 11,
alkcall version mentions, Phase 1-work bullet). Scope grew from the
task's three folded stale references + the TN-12 tracker (added as
architecture/oq-tn-12-tracker, referenced from OQ-TN-12's Blocked-on
text). taskgraph validates 13 tasks; all verification checklist items
pass.