-
Notifications
You must be signed in to change notification settings - Fork 1
backlog: P1 — factory technology inventory + OpenAI web UI addition (Aaron 2026-04-23) #165
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 16 commits
b37353d
7ab0ab9
e8b0d2d
96f612c
4b479a4
a09886e
b1ce29e
ec1787b
34dc24f
a388857
c209005
322e533
ab931f0
8a05738
5a9eec1
5538537
d1c30e6
6bea1bd
e7d90cd
0d9179c
2b9c522
9bc26af
464b4a5
c7093c3
f850d0e
f0217ea
9b7f814
de61e68
224623a
eabfa10
30ad9aa
472152b
2de7b5f
b6ec3f5
978a785
f875deb
c689aa4
e008ac0
a48a87a
137351d
e3018c3
375768a
ef221fb
aa3cf33
718253c
f81c97b
54fb96d
c101f50
56742b7
1146b00
589b749
d29474d
6d42913
c0674f9
00ac0ce
340f475
de35d5c
1968ef3
1c4522a
48e884e
9632c25
79c6d6b
09e1808
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 | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -3239,6 +3239,53 @@ within each priority tier. | |||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| **Effort:** M (1-3 days of agent research + write-up). | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| - [ ] **Factory technology inventory — first-class | ||||||||||||||||||||||||||
| support for every tech we use.** | ||||||||||||||||||||||||||
| Aaron 2026-04-23: *"don't forget to map out all our | ||||||||||||||||||||||||||
| technology so the factory has first class support for | ||||||||||||||||||||||||||
|
Comment on lines
+3331
to
+3334
|
||||||||||||||||||||||||||
| everything ... i think i saw you ad docker and postgres | ||||||||||||||||||||||||||
| and now we may be adding the openai website/ui i think | ||||||||||||||||||||||||||
|
Comment on lines
+3331
to
+3336
|
||||||||||||||||||||||||||
| we already have codex cli mapped"*. | ||||||||||||||||||||||||||
|
Comment on lines
+3331
to
+3337
|
||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| **Observed gap:** the factory uses many technologies | ||||||||||||||||||||||||||
| (Docker, Postgres, F#, .NET 10, TypeScript, bun, | ||||||||||||||||||||||||||
| Claude Code, Codex CLI, Gemini CLI, OpenAI web UI, | ||||||||||||||||||||||||||
| Playwright, Apache Arrow, Lean 4, Z3, TLA+, FsCheck, | ||||||||||||||||||||||||||
| Alloy, Semgrep, CodeQL, BenchmarkDotNet, GitHub | ||||||||||||||||||||||||||
| Actions, NuGet, and more) but no single doc | ||||||||||||||||||||||||||
| inventories them with install path, version pin, | ||||||||||||||||||||||||||
| authoritative doc URL, expert-skill cross-reference, | ||||||||||||||||||||||||||
| and TECH-RADAR ring. `docs/HARNESS-SURFACES.md` | ||||||||||||||||||||||||||
| covers agent harnesses; `docs/TECH-RADAR.md` covers | ||||||||||||||||||||||||||
| ring adoption; per-tech expert skills cover usage | ||||||||||||||||||||||||||
| depth. The missing surface is an inventory that ties | ||||||||||||||||||||||||||
| all three together per-tech. | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| **Deliverable:** `docs/FACTORY-TECHNOLOGY-INVENTORY.md` | ||||||||||||||||||||||||||
| (name TBD) with columns: Technology / Role / Install | ||||||||||||||||||||||||||
| path / Version pin / Auth-doc URL / Expert skill / | ||||||||||||||||||||||||||
| TECH-RADAR ring / Notes. | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
|
Comment on lines
+3331
to
+3357
|
||||||||||||||||||||||||||
| **Growth cadence:** living inventory updated with | ||||||||||||||||||||||||||
| each new tech adoption. Should surface cross-platform | ||||||||||||||||||||||||||
| parity status (row #48) per tech. | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
| parity status (row #48) per tech. | |
| parity status (row #51) per tech. |
Copilot
AI
Apr 23, 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.
This item says the technology inventory should surface cross-platform parity status “(row #48)”, but in docs/FACTORY-HYGIENE.md the cross-platform parity audit is row #51 (row #48 is GitHub surface triage cadence). Please update the referenced row number so it points to the correct hygiene entry.
| parity status (row #48) per tech. | |
| parity status (row #51) per tech. |
Copilot
AI
Apr 23, 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.
Cross-reference looks incorrect: FACTORY-HYGIENE row #48 is “GitHub surface triage cadence”, not the cross-platform parity audit. If the intent is to reference the cross-platform parity status, update this to the correct row number (currently row #51) or cite the row by name/link instead of number to avoid drift.
| each new tech adoption. Should surface cross-platform | |
| parity status (row #48) per tech. | |
| each new tech adoption. Should surface the | |
| FACTORY-HYGIENE cross-platform parity status per tech. |
Copilot
AI
Apr 23, 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.
Copilot
AI
Apr 23, 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: This cites a “per-user memory” file by bare filename, but there’s no resolvable path in-repo. Consider using the explicit per-user location (~/.claude/projects/<slug>/memory/<file>.md) or an in-repo mirror so readers can actually find it.
| `project_factory_technology_inventory_first_class_support_openai_playwright_hard_2026_04_23.md`). | |
| `~/.claude/projects/<slug>/memory/project_factory_technology_inventory_first_class_support_openai_playwright_hard_2026_04_23.md`). |
Copilot
AI
Apr 23, 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 (pointer integrity): This row points to a “per-user memory” file by bare filename, but that file isn’t present in-repo (and the path isn’t specified), so readers can’t follow it. Suggest either (a) capturing the essential Playwright/OpenAI UI caveats in an in-repo doc and linking that, or (b) formatting the reference with the explicit per-user location (~/.claude/projects/<slug>/memory/...) and keeping the BACKLOG row self-contained enough to stand without the private file.
| caveats (long-conversation rendering, async loading, | |
| UI-change-ongoing maintenance — per the per-user | |
| memory at | |
| `project_factory_technology_inventory_first_class_support_openai_playwright_hard_2026_04_23.md`). | |
| caveats: long-conversation rendering can degrade, | |
| async UI loading can make automation timing-sensitive, | |
| and ongoing UI changes mean the integration needs | |
| maintenance. Private supporting notes live at | |
| `~/.claude/projects/<slug>/memory/project_factory_technology_inventory_first_class_support_openai_playwright_hard_2026_04_23.md`, | |
| but this backlog row should stand without that file. |
Copilot
AI
Apr 23, 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.
P2: The PR title/description focus on the factory technology-inventory BACKLOG row + OpenAI web UI, but this PR also introduces a full AutoDream cadence policy, a new FACTORY-HYGIENE row, and a new hygiene-history ledger. Consider updating the PR title/description to reflect the full scope, or splitting into separate PRs (inventory vs AutoDream hygiene) for easier review/traceability.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,21 @@ | ||||||||||||||||||
| # AutoDream fire history | ||||||||||||||||||
|
|
||||||||||||||||||
| Per-fire ledger for FACTORY-HYGIENE row #53 (AutoDream cadenced | ||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This new history file declares itself as the ledger for Useful? React with 👍 / 👎. |
||||||||||||||||||
| consolidation — upstream Anthropic Q1 2026 + factory overlays | ||||||||||||||||||
|
Comment on lines
+3
to
+4
|
||||||||||||||||||
| A/B/C/D). Schema per row #44 (date / agent / output / link / | ||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This line points fire-history schema consumers to Useful? React with 👍 / 👎. |
||||||||||||||||||
| next-expected). | ||||||||||||||||||
|
Comment on lines
+3
to
+6
|
||||||||||||||||||
| Per-fire ledger for FACTORY-HYGIENE row #53 (AutoDream cadenced | |
| consolidation — upstream Anthropic Q1 2026 + factory overlays | |
| A/B/C/D). Schema per row #44 (date / agent / output / link / | |
| next-expected). | |
| Per-fire ledger for the AutoDream cadenced consolidation entry in | |
| `docs/FACTORY-HYGIENE.md` (upstream Anthropic Q1 2026 + factory | |
| overlays A/B/C/D). Schema: date / agent / output / link / | |
| next-expected. |
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.
Add durable-output link column to fire-history ledger
The file says it follows row #44 schema (date / agent / output / link / next-expected), but the table omits a link field entirely. Without a per-fire durable-output link, this ledger cannot provide the traceability required for cadence-history audits.
Useful? React with 👍 / 👎.
Copilot
AI
Apr 23, 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.
This fire-history table uses direct contributor/agent names (e.g., "Claude") in the Agent column, but docs are required to use role references instead (names only allowed in memory/persona/** and optionally docs/BACKLOG.md). Consider changing the Agent column to role labels (e.g., "loop agent", "manual run") and, if needed, link to an allowed persona-memory entry.
| | 2026-04-20 | Claude (manual approximation pre-row-#53) | No marker — first known fire | Pre-overlay era | MEMORY.md bootstrap — no overlay framework yet | Superseded by the 2026-04-23 policy doc | | |
| | 2026-04-23 | Claude (first row-#53 fire) | 3 days + ≥5 sessions since 2026-04-20 marker — both gates pass | Light pass: Orientation + Gather Signal (findings-only); Overlays A/B/C/D scheduled for follow-up tick | Per-user MEMORY.md is 345 lines (well over the 200-line cap); 14+ new 2026-04-23 memories not yet indexed cleanly; generic rules in per-user worth candidate migration to in-repo. Full consolidation deferred to a dedicated tick — this fire records the cadence observance and the findings. | Next fire gate-open at ≥24h AND ≥5 sessions from 2026-04-23 (likely 2026-04-26 or later). Overlay A migration candidates land on that fire or earlier opportunistic-on-touch. | | |
| | 2026-04-23 (later, same-day) | Claude (auto-loop-50 opportunistic-on-touch Overlay A) | Same session — not cadence-gated; opportunistic-on-touch per row #53 | Overlay A — first execution | Migrated `feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md` from per-user to in-repo `memory/` (PR #157). Dangling citations from `docs/FACTORY-HYGIENE.md` + `docs/research/autodream-extension-and-cadence-2026-04-23.md` resolved. Per-user source retained with "Migrated to" marker (provenance preserved). Candidate next migrations identified: `feedback_outcomes_over_vanity_metrics_goodhart_resistance.md`, `feedback_deletions_over_insertions_complexity_reduction_cyclomatic_proxy.md`, `feedback_external_signal_confirms_internal_insight_second_occurrence_discipline_2026_04_22.md`, `project_semiring_parameterized_zeta_regime_change_one_algebra_to_map_others_2026_04_22.md` — all generic factory-shape, all cited from the just-migrated file. | Next full-cadence fire at ≥24h AND ≥5 sessions from 2026-04-23. Opportunistic-on-touch continues for the 4 identified candidates as future ticks have budget. | | |
| | 2026-04-23 (later ×2, same-day) | Claude (auto-loop-52 opportunistic-on-touch Overlay A) | Same session — opportunistic-on-touch | Overlay A — second execution (working the identified queue from the prior fire) | Migrated `feedback_outcomes_over_vanity_metrics_goodhart_resistance.md` from per-user to in-repo `memory/` (PR #158). Sibling to signal-in-signal-out (same 2026-04-22 tick pair — Goodhart resistance + DSP preservation as paired disciplines). Resolves one composes-with dangling citation from the just-migrated signal-in-signal-out memory. Queue now 3 candidates (deletions-over-insertions / external-signal-confirms-internal-insight / semiring-parameterized-zeta). | Opportunistic-on-touch continues; cadenced fire still gate-open ≥2026-04-26. | | |
| | 2026-04-20 | manual run (approximation pre-row-#53) | No marker — first known fire | Pre-overlay era | MEMORY.md bootstrap — no overlay framework yet | Superseded by the 2026-04-23 policy doc | | |
| | 2026-04-23 | manual run (first row-#53 fire) | 3 days + ≥5 sessions since 2026-04-20 marker — both gates pass | Light pass: Orientation + Gather Signal (findings-only); Overlays A/B/C/D scheduled for follow-up tick | Per-user MEMORY.md is 345 lines (well over the 200-line cap); 14+ new 2026-04-23 memories not yet indexed cleanly; generic rules in per-user worth candidate migration to in-repo. Full consolidation deferred to a dedicated tick — this fire records the cadence observance and the findings. | Next fire gate-open at ≥24h AND ≥5 sessions from 2026-04-23 (likely 2026-04-26 or later). Overlay A migration candidates land on that fire or earlier opportunistic-on-touch. | | |
| | 2026-04-23 (later, same-day) | loop agent (auto-loop-50 opportunistic-on-touch Overlay A) | Same session — not cadence-gated; opportunistic-on-touch per row #53 | Overlay A — first execution | Migrated `feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md` from per-user to in-repo `memory/` (PR #157). Dangling citations from `docs/FACTORY-HYGIENE.md` + `docs/research/autodream-extension-and-cadence-2026-04-23.md` resolved. Per-user source retained with "Migrated to" marker (provenance preserved). Candidate next migrations identified: `feedback_outcomes_over_vanity_metrics_goodhart_resistance.md`, `feedback_deletions_over_insertions_complexity_reduction_cyclomatic_proxy.md`, `feedback_external_signal_confirms_internal_insight_second_occurrence_discipline_2026_04_22.md`, `project_semiring_parameterized_zeta_regime_change_one_algebra_to_map_others_2026_04_22.md` — all generic factory-shape, all cited from the just-migrated file. | Next full-cadence fire at ≥24h AND ≥5 sessions from 2026-04-23. Opportunistic-on-touch continues for the 4 identified candidates as future ticks have budget. | | |
| | 2026-04-23 (later ×2, same-day) | loop agent (auto-loop-52 opportunistic-on-touch Overlay A) | Same session — opportunistic-on-touch | Overlay A — second execution (working the identified queue from the prior fire) | Migrated `feedback_outcomes_over_vanity_metrics_goodhart_resistance.md` from per-user to in-repo `memory/` (PR #158). Sibling to signal-in-signal-out (same 2026-04-22 tick pair — Goodhart resistance + DSP preservation as paired disciplines). Resolves one composes-with dangling citation from the just-migrated signal-in-signal-out memory. Queue now 3 candidates (deletions-over-insertions / external-signal-confirms-internal-insight / semiring-parameterized-zeta). | Opportunistic-on-touch continues; cadenced fire still gate-open ≥2026-04-26. | |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,303 @@ | ||||||||||||||||||||
| # AutoDream extension + cadence — factory overlay on Anthropic's Q1 2026 feature | ||||||||||||||||||||
|
|
||||||||||||||||||||
| **Date:** 2026-04-23 | ||||||||||||||||||||
| **Status:** Research doc — preparing the factory-overlay policy | ||||||||||||||||||||
| that plugs tightly into Anthropic's AutoMemory / AutoDream | ||||||||||||||||||||
| Q1 2026 features so the factory inherits upgrades rather than | ||||||||||||||||||||
| forking its own memory-hygiene stack. | ||||||||||||||||||||
|
Comment on lines
+1
to
+7
|
||||||||||||||||||||
| **Triggered by:** The human maintainer 2026-04-23: *"continue | ||||||||||||||||||||
| our AutoDream reserach to make sure we are running it on a | ||||||||||||||||||||
| cadence and we plug in tightly with the existing claude Q1 2026 | ||||||||||||||||||||
| feature for AutoDream and AutoMemory, we should by defintion be | ||||||||||||||||||||
| an extension of theirs as it will get upgrades we want over | ||||||||||||||||||||
| time. Also they have a cadence we shuld prbably respect and | ||||||||||||||||||||
| adaopt for this in our hygene."* | ||||||||||||||||||||
| **Scope:** Factory policy — generic, reusable by any factory | ||||||||||||||||||||
| adopter on Claude Code; not project-specific to Zeta. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Why this matters | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Anthropic shipped two load-bearing features in Q1 2026: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| 1. **AutoMemory** — the persistent cross-session memory | ||||||||||||||||||||
| system itself (`MEMORY.md` index + per-fact files under | ||||||||||||||||||||
| `~/.claude/projects/<slug>/memory/`). Additive by design; | ||||||||||||||||||||
| daytime logger. | ||||||||||||||||||||
| 2. **AutoDream** — the REM-sleep-style consolidation pass | ||||||||||||||||||||
| that runs on top of AutoMemory. Subtractive / curative by | ||||||||||||||||||||
| design; nighttime hygiene. Flag-gated server-side as of | ||||||||||||||||||||
| 2026-04-19 (`tengu_onyx_plover`); the UI at `/memory` | ||||||||||||||||||||
| exposes it but the backend is off for most users. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Anthropic's documentation establishes: AutoMemory without | ||||||||||||||||||||
| AutoDream degrades after 10-15 sessions because the daytime | ||||||||||||||||||||
| log accumulates duplicates, contradictions, and stale | ||||||||||||||||||||
| relative-date references. The consolidation pass is not | ||||||||||||||||||||
| optional — it is the thing that prevents the degradation | ||||||||||||||||||||
| cliff. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The factory has been running **manual approximations** of | ||||||||||||||||||||
| AutoDream since the feature was surfaced, but without a | ||||||||||||||||||||
| documented cadence policy or a formal extension contract. | ||||||||||||||||||||
| This doc closes that gap. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## The two rules from the directive | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Rule 1 — extend, don't replace | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The factory is **by definition** an extension of Anthropic's | ||||||||||||||||||||
| memory substrate, not a parallel stack. When Anthropic ships | ||||||||||||||||||||
| an improvement to AutoMemory or AutoDream, the factory | ||||||||||||||||||||
| inherits it for free *only if* the factory has not forked the | ||||||||||||||||||||
| underlying mechanism. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Concretely: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - **Frontmatter schema stays upstream-compatible.** The | ||||||||||||||||||||
| factory may add optional keys (e.g., a future `scope:` key | ||||||||||||||||||||
| per `docs/research/memory-scope-frontmatter-schema.md`) but | ||||||||||||||||||||
| must not rename or remove the Anthropic-required keys | ||||||||||||||||||||
| (`name`, `description`, `type`, `originSessionId`) and must | ||||||||||||||||||||
| keep the type taxonomy (`user`, `feedback`, `project`, | ||||||||||||||||||||
| `reference`). | ||||||||||||||||||||
| - **Storage location stays upstream.** Per-user memory lives | ||||||||||||||||||||
| at `~/.claude/projects/<slug>/memory/`, not at a | ||||||||||||||||||||
| factory-custom path. The factory's in-repo `memory/` tree | ||||||||||||||||||||
| is a **mirror** for cross-substrate readability — it does | ||||||||||||||||||||
| not replace the per-user location; it adds a second home | ||||||||||||||||||||
| for generic content. | ||||||||||||||||||||
| - **AutoDream's four phases stay upstream.** Orientation → | ||||||||||||||||||||
| Gather Signal → Consolidation → Prune & Index. The factory | ||||||||||||||||||||
| extends by adding overlay steps *between* or *after* these | ||||||||||||||||||||
| phases, never by replacing them. | ||||||||||||||||||||
| - **When Anthropic ships automatic AutoDream** (backend un-gated), | ||||||||||||||||||||
| the factory's manual approximation retires in favour of | ||||||||||||||||||||
| the automatic run. Factory-overlay steps continue on top. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ### Rule 2 — adopt upstream cadence | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Anthropic's AutoDream cadence is **both**: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - **≥24 hours** since the last cycle, **and** | ||||||||||||||||||||
| - **≥5 sessions** since the last cycle. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The factory adopts this cadence verbatim. Running more | ||||||||||||||||||||
| frequently churns fresh memories before their | ||||||||||||||||||||
| duplicate / contradiction / staleness patterns have | ||||||||||||||||||||
| surfaced; running less frequently lets the daytime log | ||||||||||||||||||||
| degrade. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The cadence gate lives in a single cross-session marker at | ||||||||||||||||||||
| the top of `MEMORY.md`: | ||||||||||||||||||||
|
||||||||||||||||||||
| the top of `MEMORY.md`: | |
| the top of the per-user AutoMemory index file | |
| `~/.claude/projects/<slug>/memory/MEMORY.md` (not the | |
| in-repo `memory/MEMORY.md`): |
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.
Store last-run marker with time, not date-only
The policy claims a strict >=24 hours gate, but the marker format is date-only (YYYY-MM-DD), which cannot represent elapsed hours accurately. A run at 23:59 and another at 00:05 next day would appear as different dates despite only minutes passing, so the cadence check can fire too early and violate the intended upstream rule.
Useful? React with 👍 / 👎.
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.
Count unique sessions, not per-fact files, for cadence gate
The policy says the AutoDream run gate is >=24h and >=5 sessions, but this implementation substitutes >=5 new per-fact files as the session counter. Because memory files are fact-scoped, one busy session can emit multiple files and satisfy the gate early, while several sessions that produce few/no new files may never satisfy it; both cases violate the stated upstream cadence and can schedule consolidations at the wrong times.
Useful? React with 👍 / 👎.
Copilot
AI
Apr 23, 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.
P0: This doc cites memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md but also says it is “per-user until this doc migrates it”. Since the file is not present in the in-repo memory/ directory, this reference is currently non-resolvable/misleading; use an explicit per-user path (~/.claude/projects/<slug>/memory/...) or add an in-repo mirror.
| `memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md` | |
| `~/.claude/projects/<slug>/memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md` |
Copilot
AI
Apr 23, 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.
This doc references memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md as an in-repo file, but that file does not exist under memory/ in this branch (only other files refer to it). Either add the referenced memory file to memory/, or change the reference to the per-user location (~/.claude/projects/<slug>/memory/...) and adjust the surrounding text accordingly so the citation is not dangling.
| that should be in-repo per | |
| `memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md` | |
| (per-user until this doc migrates it). | |
| that should be in-repo; today it still lives at | |
| `~/.claude/projects/<slug>/memory/feedback_in_repo_preferred_over_per_user_memory_where_possible_2026_04_23.md` | |
| until this doc migrates it into the repo `memory/` tree. |
Copilot
AI
Apr 23, 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.
This doc introduces direct contributor/agent names (e.g., "Kenji") in the body. docs/AGENT-BEST-PRACTICES.md forbids name attribution in docs outside memory/persona/** (and docs/BACKLOG.md as the only doc exception). Replace names with role references (e.g., "the architect") and, if needed, point to a persona-memory file instead.
| Kenji (Architect) decides whether the promotion lands this | |
| The architect decides whether the promotion lands this |
Copilot
AI
Apr 23, 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: This section names specific individuals (“sister Elisabeth”, etc.) inside an in-repo policy doc. docs/AGENT-BEST-PRACTICES.md explicitly restricts direct contributor names in docs/skills and prefers role references (with docs/BACKLOG.md as the carve-out). Please replace these with role-based references or move the specifics to per-user memory so the repo stays compliant with the stated convention.
| rule). The factory's load-bearing list includes sister | |
| Elisabeth memory, faith memory, Harmonious Division name, | |
| Rodney persona placement, Dora persona — plus factory-scale | |
| anchors like the bootstrap-is-complete memory, the | |
| rule). The factory's load-bearing list includes core | |
| identity-anchor memory, faith-anchor memory, the | |
| Harmonious Division project-name anchor, and persona | |
| placement/roster anchors — plus factory-scale anchors | |
| like the bootstrap-is-complete memory, the |
Copilot
AI
Apr 23, 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: feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md is referenced without a resolvable in-repo path and does not exist under memory/ in this branch. Please either add the in-repo memory file, or change this to an explicit per-user path so the “composes with” reference is actionable.
| composes with | |
| `feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md`). | |
| composes with the signal-in / signal-out clean-or-better | |
| DSP discipline rule). |
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.
Require both cadence gates before scheduling a run
This rule queues AutoDream whenever the skip case is false, which means a run is scheduled when only one gate passes (e.g., >=24h but <5 sessions, or <24h but >=5 sessions). Earlier in the same doc the cadence is defined as >=24h AND >=5 sessions, so this condition weakens the gate and will trigger out-of-cadence consolidations in normal use.
Useful? React with 👍 / 👎.
Copilot
AI
Apr 23, 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.
This doc uses direct contributor/agent names (e.g., "Kenji") in normative policy text. Repo convention is to use role references in docs/skills and keep personal names only under memory/persona/<name>/ and (narrowly) docs/BACKLOG.md (see docs/AGENT-BEST-PRACTICES.md "No name attribution in code, docs, or skills"). Consider rewriting these references as roles (e.g., "the architect") or moving name-specific attribution into the appropriate persona memory/notebook.
Copilot
AI
Apr 23, 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.
This section claims a new AutoDream-specific FACTORY-HYGIENE row is added by the same PR, but this PR does not update docs/FACTORY-HYGIENE.md and there is currently no row #53 there. Either add the referenced hygiene row in this PR, or change this text (and any downstream references) to match the current row numbering/status (e.g., mark it proposed with a TBD row number).
| clock. A new FACTORY-HYGIENE row for AutoDream-specific | |
| consolidation tracks the cadence independently. | |
| clock. A follow-up should add an AutoDream-specific | |
| FACTORY-HYGIENE row (TBD row number) so consolidation | |
| cadence is tracked independently. |
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.
Remove unresolved claim about AutoDream hygiene row
The composition section says this same PR adds a new AutoDream-specific FACTORY-HYGIENE row, but this commit does not modify docs/FACTORY-HYGIENE.md, so the referenced policy surface is not actually present here. That creates an unresolvable dependency for anyone trying to follow the policy from this document.
Useful? React with 👍 / 👎.
Copilot
AI
Apr 23, 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.
This doc claims “a new AutoDream-specific [FACTORY-HYGIENE] row is added by the same PR that lands this research doc”, but this PR doesn’t modify docs/FACTORY-HYGIENE.md to add such a row. Either add the referenced hygiene row in this PR or remove/adjust this statement so it matches what actually lands.
| a new AutoDream-specific row is added by the same PR | |
| that lands this research doc) | |
| follow-up home for any AutoDream-specific hygiene row if | |
| one is added in a later change) |
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: The PR description/title focus on adding a P1 tech-inventory backlog row, but this PR also adds a large new P0 Frontier-readiness BACKLOG item plus substantial AutoDream/tick-history documentation. Please either update the PR description/title to reflect the broader scope or split these additional changes into separate PRs to keep review/rollback boundaries clear.