janpeter/scrum4me-workers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(git): node_modules-symlink uit de index halen #80

Merged
janpeter merged 1 commit from claude/determined-colden-51faf9 into main 2026-08-06 11:09:25 +02:00
Conversation 1 Commits 1 Files changed 2 +1 -2
janpeter commented 2026-08-06 09:03:03 +02:00
Owner
Copy link

Probleem

In commit 628c6e1 ("Sprint: Messages pagina", #75) is node_modules als symlink (mode 120000) in git terechtgekomen. Het doelpad is /home/agent/Projects/scrum4me-workers/node_modules — een pad uit een cloud-agent-container.

Gevolg op elke andere checkout (Mac, CI, Docker): een dangling symlink. Zodra npm install draait wordt die vervangen door een echte map, en toont git status permanent:

 D node_modules

Oorzaak dat het door de .gitignore glipte

De regel stond er wél, maar als node_modules/ — met trailing slash matcht een gitignore-pattern alleen directories. De gecommitte entry was een symlink (een blob), geen directory, dus hij werd nooit genegeerd. Slash eraf zodat ook een symlink of bestand met die naam gedekt is.

Wijziging

  • git rm --cached node_modules — pad uit de index
  • .gitignore: node_modules/ → node_modules

Andere platform-repo's gecontroleerd

Alle gecommitte symlinks (ls-files -s, mode 120000) op origin/main:

Repo Symlinks op main
Scrum4Me (web) geen
scrum4me-mcp geen
scrum4me-shared alleen AGENTS.md → CLAUDE.md (bedoeld, relatief pad)

Alleen scrum4me-workers was getroffen.

Verify

  • git ls-files -s node_modules → leeg
  • npm run verify → groen (lint + typecheck + 114 test files, 797 tests passed, 10 skipped)

Niet zelf gemerged — klaar voor review.

🤖 Generated with Claude Code

## Probleem In commit 628c6e1 ("Sprint: Messages pagina", #75) is `node_modules` als **symlink** (mode `120000`) in git terechtgekomen. Het doelpad is `/home/agent/Projects/scrum4me-workers/node_modules` — een pad uit een cloud-agent-container. Gevolg op elke andere checkout (Mac, CI, Docker): een dangling symlink. Zodra `npm install` draait wordt die vervangen door een echte map, en toont `git status` permanent: ``` D node_modules ``` ## Oorzaak dat het door de .gitignore glipte De regel stond er wél, maar als `node_modules/` — **met trailing slash matcht een gitignore-pattern alleen directories**. De gecommitte entry was een symlink (een blob), geen directory, dus hij werd nooit genegeerd. Slash eraf zodat ook een symlink of bestand met die naam gedekt is. ## Wijziging - `git rm --cached node_modules` — pad uit de index - `.gitignore`: `node_modules/` → `node_modules` ## Andere platform-repo's gecontroleerd Alle gecommitte symlinks (`ls-files -s`, mode 120000) op `origin/main`: | Repo | Symlinks op main | |---|---| | Scrum4Me (web) | geen | | scrum4me-mcp | geen | | scrum4me-shared | alleen `AGENTS.md` → `CLAUDE.md` (bedoeld, relatief pad) | Alleen scrum4me-workers was getroffen. ## Verify - `git ls-files -s node_modules` → leeg - `npm run verify` → groen (lint + typecheck + 114 test files, 797 tests passed, 10 skipped) Niet zelf gemerged — klaar voor review. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(git): node_modules-symlink uit de index halen
All checks were successful
CI / Verify (pull_request) Successful in 1m51s
Details
9929fc05c4 
Commit 628c6e1 committe node_modules als symlink (mode 120000) naar
/home/agent/Projects/scrum4me-workers/node_modules — een pad uit een
cloud-agent-container. Op elke andere checkout (Mac, CI, Docker) is dat
een dangling symlink, en na `npm install` toont `git status` permanent
" D node_modules".

De .gitignore-regel `node_modules/` (met slash) matcht alleen directories,
dus een symlink met die naam werd nooit genegeerd — vandaar dat hij kon
binnenglippen. Slash eraf zodat ook een symlink/bestand gedekt is.

Andere platform-repo's gecontroleerd (Scrum4Me, scrum4me-mcp,
scrum4me-shared): geen node_modules-symlink op origin/main; alleen de
bedoelde AGENTS.md -> CLAUDE.md in scrum4me-shared.

Verify: `git ls-files -s node_modules` leeg, `npm run verify` groen
(114 test files, 797 tests).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-08-06 09:06:16 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen findings.

De PR verwijdert een lokale absolute node_modules-symlink uit de index en past .gitignore aan naar node_modules, waardoor zowel een directory als een symlink/bestand met die naam genegeerd blijft. Dit is een kleine, passende repository-hygiënefix zonder code-, architectuur-, test- of docs-impact.

# APPROVED geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen findings. De PR verwijdert een lokale absolute `node_modules`-symlink uit de index en past `.gitignore` aan naar `node_modules`, waardoor zowel een directory als een symlink/bestand met die naam genegeerd blijft. Dit is een kleine, passende repository-hygiënefix zonder code-, architectuur-, test- of docs-impact.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-workers!80
No description provided.