chore(git): remove orphan gitlinks (.claude, bms-monorepo, evo-ai-docs)

[daniloleonecarneiro]

Summary

CI quebra com:

fatal: No url found for submodule path '.claude' in .gitmodules
fatal: No url found for submodule path 'bms-monorepo' in .gitmodules
fatal: No url found for submodule path 'evo-ai-docs' in .gitmodules
Error: The process '/usr/bin/git' failed with exit code 128

Causa: no commit 73646ae (chore(submodules): update submodule commits and add new submodules), três paths foram adicionados ao índice como gitlinks (mode 160000) sem entries correspondentes em .gitmodules:

Path	SHA	Em .gitmodules?
.claude	e8b141d	❌
bms-monorepo	374e13c	❌
evo-ai-docs	9cf73e9	❌

Em todo checkout/CI o git submodule update --init aborta no primeiro path órfão. .gitignore já tinha .claude/* (ignora conteúdo, não a entrada gitlink na árvore); bms-monorepo/ e evo-ai-docs/ nunca foram ignorados.

Fix

• git rm --cached .claude bms-monorepo evo-ai-docs — remove os 3 gitlinks órfãos do índice.
• .gitignore: adicionar .claude, .claude/, bms-monorepo/ e evo-ai-docs/ para impedir que sejam re-adicionados.

.claude/ é config local per-developer do Claude Code. bms-monorepo/ e evo-ai-docs/ são clones-irmãos locais que devem ficar fora deste superproject — se forem virar submodules de verdade, isso é mudança separada com entries em .gitmodules.

Test plan

• git ls-tree HEAD | awk '$2=="commit"' — só lista os 10 submodules registrados em .gitmodules.
• git status no working tree limpo — .gitignore cobre os 3 diretórios.
• CI verde após merge — git submodule update --init não levanta mais o fatal: No url found.
• Devs com .claude/, bms-monorepo/, evo-ai-docs/ locais não veem como untracked após pull.

[sourcery-ai]

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Removes an orphaned .claude gitlink that was causing submodule-related CI failures and tightens .gitignore rules so .claude as a directory or file is never tracked while still ignoring any contents under it.

File-Level Changes

Change	Details	Files
Remove orphan .claude gitlink from the index and harden ignore rules so local Claude config is never tracked or treated as a submodule.	Delete the .claude gitlink entry from version control so Git no longer treats it as a submodule without a matching .gitmodules entry. Update .gitignore to ignore .claude and .claude/ in addition to .claude/*, preventing the directory itself (and its contents) from being tracked if present locally.	.claude .gitignore

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[dpaes]

Code review — EVO-1241 prerequisite (orphan gitlinks)

Code is correct. Removes the .claude, bms-monorepo, evo-ai-docs gitlinks (mode 160000, no .gitmodules entry, added in 73646ae) and ignores them, fixing git submodule update --init across CI/checkout. Verified: on this branch Lint Dockerfiles and Validate Docker Compose both pass — and those same jobs fail on #101 purely because of these orphan gitlinks. So this PR unblocks the whole monorepo CI.

🟢 LOW — .gitignore ends up with .claude, .claude/ and .claude/* (redundant, harmless).

⚠️ The real gate here is process, not code: the card flags "Necessário conversar com o Davidson antes". This touches the submodule structure — if bms-monorepo / evo-ai-docs are meant to become real submodules, this PR hides them. Not a review call to make unilaterally.

No verdict — awaiting Davidson's decision.

[dpaes]

Code review — EVO-1241 — Round 2 (delta vs the 03:03 review)

No commits pushed since round 1 (head still chore(git): remove orphan bms-monorepo / evo-ai-docs gitlinks, 01:55). My round-1 read stands: the code is correct — it removes the orphan 160000 gitlinks (.claude, bms-monorepo, evo-ai-docs, introduced by 73646ae with no .gitmodules entry) and unbreaks git submodule update --init. I re-confirmed this branch makes Lint Dockerfiles and Validate Docker Compose pass, which is what unblocks the rest of the stack's CI.

The real gate here is process, not code: the card flags "Necessário conversar com o Davidson antes" — if bms-monorepo/evo-ai-docs are meant to become real submodules, this PR hides them instead. That sign-off is the blocker.

🟢 L-4 (.gitignore lists .claude, .claude/, .claude/* — redundant, harmless) — unchanged nit.

No verdict from me — awaiting Davidson's sign-off; consolidated status on the Linear card.

[dpaes]

✅ Code LGTM (round-2 re-review). The gitlink cleanup is correct — it removes the orphan 160000 entries (.claude, bms-monorepo, evo-ai-docs) with no .gitmodules entry and unbreaks git submodule update --init; verified it makes Lint Dockerfiles and Validate Docker Compose pass, which unblocks the rest of the stack's CI.

⚠️ Not approving/merging yet — by design. The card flags "Necessário conversar com o Davidson antes": if bms-monorepo/evo-ai-docs are meant to become real submodules, this PR hides them instead. That sign-off is the gate, and this is the first merge in the chain (#102 → #90 → #3 → submodule bump → #101). Holding the formal approve + merge until Davidson clears it.

[dpaes]

Update: the reviewer (@daniel.paes) has authorized proceeding with the full merge chain, so the hold above is lifted. Merging now as the first link (#102 → #90 → #3). The submodule-pointer bump and #101 are intentionally left for the release coordination (not part of this pass).