christian merged pull request christian/claude-playground#2 2026-10-21 00:58:55 +02:00
Fix duplicate ids, validate CLI input, harden load/save
christian pushed to main at christian/claude-playground 2026-10-21 00:58:55 +02:00
14a5774d32 Merge pull request 'Fix duplicate ids, validate CLI input, harden load/save' (#2) from fix/review-findings into main
130ffe0d13 Drop the lock file: too much machinery for a personal CLI
dab492da1e Lock during updates, repair duplicate ids, keep file identity on save
f09d20cb84 Address review round 3: id limit, private temp files, canonical ids
089fc94e16 Address review round 2: duplicate ids, read-only files, dangling links, safe ids
Compare 7 commits »
christian merged pull request christian/claude-playground#1 2026-10-21 00:58:41 +02:00
Document branch-and-PR workflow; never write to main
christian pushed to main at christian/claude-playground 2026-10-21 00:58:41 +02:00
696049b3b1 Merge pull request 'Document branch-and-PR workflow; never write to main' (#1) from docs/branch-pr-workflow into main
f666bdf6b3 Give the PR workflow a hard stop
ed197eeeb1 Address review round 3: safe API snippets, PR opened once, pinned review
fa45c58c0c Address review round 2: no-track branches, honest enforcement, safer API calls
962c1aa9a1 Address review: fresh base, named review target, round cap, Gitea API
Compare 6 commits »
christian merged pull request christian/claude-playground#3 2026-10-21 00:58:31 +02:00
Protect main: git hooks + Claude Code deny rules
christian pushed to main at christian/claude-playground 2026-10-21 00:58:31 +02:00
2088568b5e Merge pull request 'Protect main: git hooks + Claude Code deny rules' (#3) from chore/deny-push-main into main
480ed6a8b2 Install hooks into the repo's own hooks dir, ignoring core.hooksPath
24541f0602 Install hooks by copying into .git/hooks instead of core.hooksPath
8b08c7b27a Block pushes and commits to main with git hooks
183e44db62 Deny Claude Code pushes to main
Compare 5 commits »
christian pushed to docs/branch-pr-workflow at christian/claude-playground 2026-10-21 00:57:15 +02:00
f666bdf6b3 Give the PR workflow a hard stop
christian pushed to fix/review-findings at christian/claude-playground 2026-10-21 00:56:20 +02:00
130ffe0d13 Drop the lock file: too much machinery for a personal CLI
christian pushed to chore/deny-push-main at christian/claude-playground 2026-10-21 00:55:53 +02:00
480ed6a8b2 Install hooks into the repo's own hooks dir, ignoring core.hooksPath
christian commented on pull request christian/claude-playground#3 2026-10-21 00:53:46 +02:00
Protect main: git hooks + Claude Code deny rules

Needs fixes (stopping here: handing off to the human) Automated self-review by Claude; not an independent review.

Round 3 (after 24541f0). The round-2 findings are fixed, including the hook…

christian commented on pull request christian/claude-playground#2 2026-10-21 00:53:45 +02:00
Fix duplicate ids, validate CLI input, harden load/save

Needs fixes (stopping here: handing off to the human) Automated self-review by Claude; not an independent review.

Round 4 (after dab492d). The round-3 findings are fixed. All 9 new findings…

christian pushed to docs/branch-pr-workflow at christian/claude-playground 2026-10-21 00:52:04 +02:00
ed197eeeb1 Address review round 3: safe API snippets, PR opened once, pinned review
christian pushed to fix/review-findings at christian/claude-playground 2026-10-21 00:49:50 +02:00
dab492da1e Lock during updates, repair duplicate ids, keep file identity on save
christian pushed to chore/deny-push-main at christian/claude-playground 2026-10-21 00:45:05 +02:00
24541f0602 Install hooks by copying into .git/hooks instead of core.hooksPath
christian pushed to fix/review-findings at christian/claude-playground 2026-10-21 00:41:45 +02:00
f09d20cb84 Address review round 3: id limit, private temp files, canonical ids
christian pushed to chore/deny-push-main at christian/claude-playground 2026-10-21 00:40:41 +02:00
8b08c7b27a Block pushes and commits to main with git hooks
christian commented on pull request christian/claude-playground#2 2026-10-21 00:35:29 +02:00
Fix duplicate ids, validate CLI input, harden load/save

Needs fixes Automated self-review by Claude; not an independent review.

Round 3 (after 089fc94). Round 2's correctness findings are fixed. New findings, most important…

christian commented on pull request christian/claude-playground#1 2026-10-21 00:34:35 +02:00
Document branch-and-PR workflow; never write to main

Needs fixes Automated self-review by Claude; not an independent review.

Round 3 (after fa45c58). Round 2's findings are addressed. New findings, most important…

christian pushed to fix/review-findings at christian/claude-playground 2026-10-21 00:34:21 +02:00
089fc94e16 Address review round 2: duplicate ids, read-only files, dangling links, safe ids
christian pushed to docs/branch-pr-workflow at christian/claude-playground 2026-10-21 00:32:52 +02:00
fa45c58c0c Address review round 2: no-track branches, honest enforcement, safer API calls