Skip to content

docs(bc-cmtr): trim CLAUDE.md and add terraform and yaml to Serena - #471

Merged
gitrgoliveira merged 1 commit into
mainfrom
worktree-bridge-cse_01WKu23VAJUZQ91ZA9nxeWns
Oct 12, 2026
Merged

gitrgoliveira merged 1 commit into
mainfrom
worktree-bridge-cse_01WKu23VAJUZQ91ZA9nxeWns

Conversation

@gitrgoliveira

Copy link
Copy Markdown
Owner

Summary

Folds two pieces of uncommitted work, found while reaping worktrees, into one PR:

  • CLAUDE.md trim (left uncommitted in the doctor-claude-md-trim worktree), applied to today's CLAUDE.md:
    • the tournament-data/ file tree is replaced by a pointer to docs/architecture/data-model.md, which already owns the layout (every file in the tree, history/ included, is described there);
    • the mobile-app runtime-defaults table is condensed to one paragraph that keeps the non-obvious facts (WriteTimeout 0 for SSE, the 64 MB import cap, SSE_MAX_CLIENTS); the exact values stay in the constants it links;
    • four areas-to-watch notes that gocloc can re-derive are dropped; "No top-level interfaces" stays, as it reads on main today;
    • the Testing Conventions section is dropped (table-driven tests, testify, defer cleanup, the test factories): the same list remains in .github/copilot-instructions.md.
  • Serena: .serena/project.yml adds the terraform and yaml language servers (left uncommitted in the bridge-cse_01N8XTrY worktree), and CLAUDE.md names the new list. The repo has 6 tracked .tf files.

Why

CLAUDE.md is loaded into every model call of every session and subagent; this removes about 3.5 KB (243,451 → 239,953 bytes) without dropping a rule. Both pieces of work would otherwise have been lost when their worktrees were removed.

Files changed

File What
CLAUDE.md data tree → pointer; runtime table → paragraph; four areas-to-watch notes and Testing Conventions removed; Serena language list updated
.serena/project.yml terraform and yaml added to language_servers

Test plan

  • make go/test passes (lint + security scan + tests)
  • New/updated unit tests cover the change: n/a, instructions and Serena config only
  • Manual browser verification: n/a, no web-mobile/ or web/ change
  • Screenshots: n/a, no UI change
  • Docs under docs/: n/a; docs/architecture/data-model.md already covers every file the removed tree listed (checked by name)
  • Docs screenshots and videos: none changed
  • No new console errors or warnings: n/a, no UI change
  • A fresh Serena instance on this branch reports languages go, typescript, scss, terraform, yaml; terraform-ls started in 0.3 s and the YAML server in 2.3 s (its one logged "Error: method 'workspace/configuration' not handled" is the server noting an optional request Serena does not answer; startup completes)

Closes bc-cmtr

🤖 Generated with Claude Code

Fold two pieces of uncommitted work into one change:
- CLAUDE.md: point to docs/architecture/data-model.md instead of repeating
  the tournament-data tree, condense the mobile-app runtime-defaults table,
  drop four areas-to-watch notes gocloc can re-derive, and drop the Testing
  Conventions section.
- .serena/project.yml: add the terraform and yaml language servers; CLAUDE.md
  names the new list.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 12, 2026 20:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@gitrgoliveira
gitrgoliveira merged commit f22be32 into main Oct 12, 2026
13 checks passed
@gitrgoliveira
gitrgoliveira deleted the worktree-bridge-cse_01WKu23VAJUZQ91ZA9nxeWns branch October 12, 2026 20:22
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.

2 participants