Repository navigation
docs(plan): record merge dates for finished DS/NAV rows - #1649
Conversation
|
@Cuarroc Dieser PR hat einen Konflikt mit main und kann nicht in die Merge-Queue. Bitte main hineinmergen (kein Rebase, kein Force-Push) und den Konflikt loesen; das Label |
|
@Cuarroc Dieser PR hat einen Konflikt mit main und kann nicht in die Merge-Queue. Bitte main hineinmergen (kein Rebase, kein Force-Push) und den Konflikt loesen; das Label |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f86d808db6
ℹ️ 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".
| | DS-X-04 | Fabrik-Orb und sechs Zustandswort-Aliase gemäß D4 | Root | Entscheidung D4 vom 15.10. (`docs/decisions.md`) | Server | B | Erledigt: PR #1603; statischer Orb, Aliase und Kontrastprüfung geliefert; Bildschirmeinbindung und Bewegung bleiben DS-X-23/24 beziehungsweise DN-03. | sieben Orb-Formen je mit eigenem Zeichen, sechs Aliase ohne zweites Vokabular; Belege im Report von #1603: Fabrik-/Zustands-/i18n-Tests, red-first, `node scripts/contrast-check.mjs` (740 Paarungen) und eigene prepush je Exit 0; zwei Stufe-B-Reviews fremder Familien; Root-Abnahme | – | | ||
| | DS-X-14 | Komponenten-Register und Abdeckungstest: `src/design/registry.ts` führt jede exportierte Designkomponente einmal (Familie, Varianten, Zustände, Bedienhilfe, Textquelle), `registry.coverage.test.ts` wird rot, wenn eine fehlt oder ein Eintrag keine Komponente mehr trifft | Implementierer · Claude; Root nimmt ab | DS-Blueprint-Schnitt 16.10. (DS-X-14) | Server | B | Offen: PR #1656 (Entwurf, Reviews ausstehend); 25 Komponenten gelistet, auch `hall/` wird gescannt (dort noch keine .tsx, DS-H-04 trägt seinen Eintrag selbst ein); Galerie DS-X-15 und Geschichten DS-X-16/17 lesen später dieses Register | Abdeckungstest rot→grün; `vitest run src/design`, typecheck, `plan-check --offline` und prepush Exit 0; zwei Stufe-B-Reviews fremder Nicht-Anthropic-Familien | – | | ||
| | DS-T-03 | Vertrauensfehler P1-b/e: die Kartenlampe sagt, was bewiesen ist (`lampFor.ts`, `lamps/texts.ts`, `AgentCard.tsx`, `FlowTab.tsx`) | Implementierer · Claude; Root nimmt ab | Lampenform aus DS-T-02 (#1610) | Server | B | Erledigt: #1624; unbekannte Spalte oder unbekannter Worker-Status zeigt auf Karte und Fabrik-Knoten die graue Unbekannt-Lampe statt „Läuft“, die grüne Lampe heißt „Test bestanden“ mit dem Alter des Testlaufs; welcher Stand getestet wurde, bleibt ein späteres Nahtpaket | fünf Test-First-Tests rot an der Merge-Base, grün am Kopf (`red-first.sh` Exit 0); typecheck, `node scripts/contrast-check.mjs` und prepush Exit 0; zwei Stufe-B-Reviews verschiedener Nicht-Autor-Familien: Subscription + OpenRouter; Root-Abnahme | – | | ||
| | DS-T-03 | Vertrauensfehler P1-b/e: die Kartenlampe sagt, was bewiesen ist (`lampFor.ts`, `lamps/texts.ts`, `AgentCard.tsx`, `FlowTab.tsx`) | Implementierer · Claude; Root nimmt ab | Lampenform aus DS-T-02 (#1610) | Server | B | Erledigt: #1624 (merged 2026-10-16); unbekannte Spalte oder unbekannter Worker-Status zeigt auf Karte und Fabrik-Knoten die graue Unbekannt-Lampe statt „Läuft“, die grüne Lampe heißt „Test bestanden“ mit dem Alter des Testlaufs; welcher Stand getestet wurde, bleibt ein späteres Nahtpaket | fünf Test-First-Tests rot an der Merge-Base, grün am Kopf (`red-first.sh` Exit 0); typecheck, `node scripts/contrast-check.mjs` und prepush Exit 0; zwei Stufe-B-Reviews verschiedener Nicht-Autor-Familien: Subscription + OpenRouter; Root-Abnahme | – | |
There was a problem hiding this comment.
Mark the already-merged DS-X-14 row done
Before completing this plan sync, update the immediately preceding DS-X-14 row: the reviewed parent already contains merge commit aaeb414 for PR #1656, but that row still says the PR is an open draft with reviews outstanding. This leaves the canonical plan reporting a false current state even as neighboring rows receive their merge status, contrary to the repository requirement that a change leave no other plan row stale.
AGENTS.md reference: AGENTS.md:L121-L121
Useful? React with 👍 / 👎.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Root auto-queue: head f86d808 unchanged since review-ok, all checks green. @Mergifyio queue |
|
@Mergifyio queue |
☑️ This pull request is already queued |
Merge Queue Status
This pull request spent 13 minutes 56 seconds in the queue, including 13 minutes 34 seconds running CI. Required conditions to merge
|
Dieser Pull Request trägt in
docs/PLAN.mddie Merge-Daten für fertige Zeilen nach (DS-T, DS-H-00 bis DS-H-03, NAV-0 bis NAV-2b, PLAN-GATE, ROADMAP-1c-1) und setzt DS-H-02b auf Erledigt. Dazu wird ein zitierter Oberflächentext in DS-T-01 umformuliert, damitplan:checkdie Zeile nicht als Widerspruch liest. Es ändert sich nur Text in einer Dokumentationsdatei, kein Code.Report
Was sich geändert hat (English):
docs/PLAN.mdonly, 17 lines (after merging main at 92830db). The status cells of rows whose PR is merged now readErledigt: #<n> (merged 2026-10-16)(PRs #1609, #1610, #1647, #1648, #1652, #1657, #1659, #1623, #1624, #1625, #1629, #1630, #1633, #1636, #1637, #1641, #1642). In DS-T-01 the quoted UI string was reworded so the parser no longer reads "erledigt" together with "offen" as a contradiction. Goals, acceptance, owners and row set are untouched; no row was deleted.STAND.md,KNOWN_ISSUES.mdand thedocs/plan/v2.0/plan.mdstatus line already name the latest releasev1.6.6(package.json on main: 1.6.6), so they are unchanged.Evidence (base
origin/mainf42baa3):npm run plan:checkbefore: Exit 1, 1 finding (DS-T-01 fix(design): DS-T-01 main view tells unknown from empty #1623); after: Exit 0, 0 findings.npm run plan:check -- --offline: Exit 0.gh pr view <n> --json mergedAt(all state MERGED).plan-check,doc-links,typecheck,cargo-check): green, Exit 0. Push withPA_PREPUSH=light, verified withgit ls-remote(head 849abaf).NICHT ABGEDECKT: the
#[cfg(unix)]tests and the Linux arms of clippy (docs-only change); fullprepushlane andnpm run test:hqwere not run (no "✅" prefix added, so the hq status parser is not affected); GitHub list limit 300 inplan:check(older PRs not fully checked).Review: tier C (docs only), the gates suffice.
Open: DS-H-03, DS-H-04 and DS-X-14 are deliberately left as main has them (their own package PRs edit those rows; they get dates in the next sync).
plan:checkonline therefore shows 3 findings not caused by this diff: DS-X-14 (#1656 merged, row left on purpose), #1662 without plan row, and PLAN-SYNC matching this PR by branch name. Offline: 0.🤖 Generated with Claude Code
Root acceptance (head f86d808, 2026-10-16 ~23:00 CEST)
Tier C (docs/PLAN.md only, no runtime effect): gates suffice, no external review required (AGENTS.md rule 5). Root check:
git diff origin/main...f86d808dtouches 17 status cells only (DS-T-01..05, DS-H-00/01/02a/02b, NAV-0/1a/2a/2b/2c, PLAN-GATE, ROADMAP-1c-1/1c-2); rows DS-H-03, DS-H-04, DS-X-14 left as on main by agreement; every named PR verified MERGED. Known: the running DS-X-15 job inserts its row next to DS-T-03 and will need a merge of main after this PR (Root dispatches it).review-okbound to f86d808.