docs: accept ADR-0021 — monorepo consolidation decision with ecosystem and extraction analysis#22
Merged
JustAGhosT merged 2 commits intomasterfrom Mar 21, 2026
Conversation
… analysis Co-authored-by: JustAGhosT <5531814+JustAGhosT@users.noreply.github.com> Agent-Logs-Url: https://github.com/JustAGhosT/codeflow-engine/sessions/8069b331-8ba2-46d3-88c9-3aa6120c0e20
Copilot
AI
changed the title
[WIP] Discussion on reintegrating codeflow repositories into a monorepo
docs: accept ADR-0021 — monorepo consolidation decision with ecosystem and extraction analysis
Mar 21, 2026
Copilot stopped work on behalf of
JustAGhosT due to an error
March 21, 2026 00:34
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
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.
ADR-0021 was stuck in "Proposed" with a recommendation to split into 3–4 repos, while the actual migration had already gone the opposite direction (all
codeflow-*repos imported into this monorepo). This closes the loop by accepting the monorepo decision and answering the open ecosystem questions.ADR-0021 (
docs/adr/0021-repository-structure.md)Rewritten from "Proposed / split recommended" → Accepted / monorepo confirmed.
Component coupling table —
desktop/andvscode-extension/integrate withengine/via HTTP/WebSocket API only (no direct imports), so independent release cadences are preserved inside the monorepo without extra tooling.Why monorepo is correct now — most split repos were already archived; re-splitting recreates maintenance burden with no payoff; path-aware CI (
dorny/paths-filter) eliminates the full-build penalty; cross-repo coordination cost is unjustified at current team size.Ecosystem fit — no
phoenixvcreferences exist anywhere in the codebase. Documents each component's role forjustaghost/*/phoenixvc/*consumers and defines the integration path if a shared org infra repo is ever created.Future extraction candidates (with explicit trigger conditions, not speculative):
website/orchestration/bootstrap/justaghost/*orphoenixvc/*infra repo is establishedSupporting docs
docs/adr/README.md— ADR-0021 entry updated to Accepted ⭐; "Future Considerations" section updated.MIGRATION_PLAN.md— Phase 1 marked complete (adds items completed since the plan was written); pending work renamed Phase 2; Future Extraction Candidates table added with link back to the ADR.Original prompt
📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.