Skip to content

Remove broken Agentic-Flow submodule#503

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/remove-broken-agentic-flow-submodule-2026-05-08-23-52
May 8, 2026
Merged

Remove broken Agentic-Flow submodule#503
NagyVikt merged 1 commit intomainfrom
agent/codex/remove-broken-agentic-flow-submodule-2026-05-08-23-52

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

@NagyVikt NagyVikt commented May 8, 2026

Automated by gx branch finish (PR flow).

The prior repair added .gitmodules for examples/agentic-flow, but the pinned upstream checkout contains its own broken nested submodule metadata. A recursive GitHub checkout cleanup can still fail inside that source. Colony only uses Agentic-Flow as documentation-only research evidence, so keeping a live submodule is unnecessary release risk.

Constraint: docs/agentic-bridge.md explicitly says Agentic-Flow must not become vendored source or a runtime dependency.

Rejected: Keep the submodule with top-level metadata | recursive cleanup still fails when the submodule checkout is present locally or in any future submodule-enabled checkout.

Rejected: Vendor selected Agentic-Flow files | violates the documentation-only source-use rule.

Confidence: high

Scope-risk: narrow

Directive: Cite upstream Agentic-Flow URLs in docs; do not re-add examples/agentic-flow as a gitlink unless upstream nested submodule metadata is valid and CI cleanup is proven.

Tested: git ls-files -s has no 160000 entries; git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"; git diff --check

Co-authored-by: OmX <omx@oh-my-codex.dev>
@NagyVikt NagyVikt merged commit 2066ba8 into main May 8, 2026
@NagyVikt NagyVikt deleted the agent/codex/remove-broken-agentic-flow-submodule-2026-05-08-23-52 branch May 8, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant