Skip to content

chore(config): remove per-repo persona-write-guard (AD-44 Revision) - #37

Merged
GmanFooFoo merged 1 commit into
mainfrom
maschin/2026-08-06-pwg-removal
Aug 6, 2026
Merged

chore(config): remove per-repo persona-write-guard (AD-44 Revision)#37
GmanFooFoo merged 1 commit into
mainfrom
maschin/2026-08-06-pwg-removal

Conversation

@GmanFooFoo

@GmanFooFoo GmanFooFoo commented Aug 6, 2026

Copy link
Copy Markdown
Member

Entfernt .claude/settings.json. Der Persona-Write-Guard lebt laut AD-44 Revision 2026-07-30 nur noch global in ~/.claude/settings.json.

Warum das mehr ist als Aufräumen: die hier liegende Kopie trägt einen Roster, der vor der Existenz der Personas ozzy und wendy eingefroren wurde. Projekt-Settings mergen mit den globalen, und der blockierende Hook gewinnt — ein wendy/…- oder ozzy/…-Push wird in diesem Repo also von einem Artefakt abgelehnt, das die AD für pensioniert hält (dev-environment #115).

Datei ist byte-identisch mit 34 weiteren und enthält ausschließlich {hooks: {PreToolUse}} — kein fremder Inhalt geht verloren.

Welle 1 von 2 (public zuerst). Sammelticket: neckarshore-ai/neckarshore-planning#1417

Summary by CodeRabbit

  • Chores
    • Removed an internal command-validation configuration.
    • No user-facing functionality or behavior was changed.

…26-07-30)

The guard lives only in the global ~/.claude/settings.json. This repo's copy
carries a roster frozen before the ozzy and wendy personas existed, so a
wendy/... or ozzy/... push is denied here by an artifact the AD considers
retired (dev-environment #115).

Measured 2026-08-04, re-verified 2026-08-06: 36 repos carried it, not the
three the AD's Affects-list named. Wave 1 of 2 (public repos first).

Refs: neckarshore-ai/neckarshore-planning#1417

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
md-viewer Building Building Preview Aug 6, 2026 10:03am

Request Review

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

The pull request deletes .claude/settings.json, including its PreToolUse Bash hook for restricting Git push and pull-request commands.

Changes

Cohort / File(s) Summary
Command guard removal
.claude/settings.json
Deletes the command interception hook that validated branches and blocked unsanctioned push or pull-request operations.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the removal of the repository-level persona-write-guard configuration, which is the main change.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch maschin/2026-08-06-pwg-removal

Comment @coderabbitai help to get the list of available commands.

@GmanFooFoo GmanFooFoo added the skip-changelog Deliberate, auditable bypass of the AD-42 changelog gate (docs/tooling/refactor-only PRs) label Aug 6, 2026
@GmanFooFoo GmanFooFoo closed this Aug 6, 2026
@GmanFooFoo GmanFooFoo reopened this Aug 6, 2026
@GmanFooFoo
GmanFooFoo merged commit 32a2ea7 into main Aug 6, 2026
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Deliberate, auditable bypass of the AD-42 changelog gate (docs/tooling/refactor-only PRs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant