Repository navigation
docs(adr): fix stale 0006/0031, add 0032 (sub-workflows) + 0033 (DI), Proposed-backlog roadmap note - #92
Merged
Conversation
…dmap note ADR review follow-ups: - 0006 (LLM provider): the "static at startup" registry consequence was stale — 0031 made provider construction dynamic (per-provider factories resolving secret/credential refs per environment). Updated the consequence and retargeted the superseded ADR-0008 references to ADR-0031. - 0031: mark Phase 2 credential objects as shipped (domain, repo, migration 000011, /v1/credentials CRUD, fuse credentials CLI, and the three reference forms over the reserved cred/<id>/<field> namespace); only Infisical remains scheduled. Note SourceCredential. - New 0032: sub-workflow composition (child workflows as first-class instances linked by a durable SubWorkflowRef; sync/async; journal + HA) — previously undocumented. - New 0033: dependency injection & app composition with uber-go/fx modules — previously undocumented. - README: index rows for 0032/0033 + a roadmap note clarifying the Proposed agent-capabilities backlog (0026-0030) and that browser automation (0025) is a separate stream. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-ups from an ADR review of the full corpus (31 ADRs).
Stale fixes
process" consequence was stale: ADR-0031 made provider construction dynamic (per-provider
llm.ProviderFactoryclosures resolving{{secret:}}/{{credential:}}per environment). Updatedthe consequence and retargeted the two references to the now-superseded ADR-0008 → ADR-0031.
scheduled," but Phase 2 credentials shipped (domain, repo, migration
000011,/v1/credentialsCRUD,fuse credentialsCLI, and the three reference forms over the reservedcred/<id>/<field>namespace). Marked shipped; only Infisical remains scheduled. NotedSourceCredential.New ADRs (documenting existing, costly-to-reverse decisions)
SubWorkflowRef(sync/async, journalsubworkflow:*, HA cross-node completion). Previously onlymentioned in passing.
fx.Module/AllModulescomposition root, driver selection, optional deps, lifecycle, and partial graphs for CLI/tests.
ADR-0002 covers the ergo runtime, not app assembly.
Index hygiene
ProposedADRs 0026–0030are a cohesive agent-capabilities backlog (none implemented), and 0025 (browser automation) is
a separate parallel stream.
Docs-only; all intra-ADR links verified to resolve.
🤖 Generated with Claude Code