Skip to content

fix(quality): skip provenance archives in brand-consistency scan#494

Merged
neuron7xLab merged 1 commit intomainfrom
fix/brand-consistency-archive-allowlist
Apr 27, 2026
Merged

fix(quality): skip provenance archives in brand-consistency scan#494
neuron7xLab merged 1 commit intomainfrom
fix/brand-consistency-archive-allowlist

Conversation

@neuron7xLab
Copy link
Copy Markdown
Owner

Problem

`tools/quality/brand_consistency.py` reports 66 violations across 4 file trees that contain legacy brand names (`TradePulse`, `HydroBrain`, `neurotrade`, `NeuroPro`) by design — they document the old-repo → GeoSync rebrand and lose audit value if scrubbed.

Fix

Added four directory prefixes to `SKIP_PREFIXES` (same pattern as existing `docs/archive`):

  • `.claude/archive/` — rebrand provenance ledgers (ACTION_RESULT_ACCEPTOR_LEDGER, OLD_REPO_SALVAGE_LEDGER)
  • `tests/archive/` — tests that exercise the archive ledger schemas
  • `tools/archive/` — tooling that compares old-repo to new-repo by name
  • `.claude/worktrees/` — ephemeral Claude Code agent worktrees, not repo source

Evidence

  • Before: `✗ brand-consistency: 66 violation(s) across 3851 scanned files`
  • After: `✓ brand-consistency: clean (5291 files scanned, 91 allowlisted)`
  • Self-tests: 24/24 pass

What this is NOT

  • Not adding individual allowlist entries — the prefix mechanism is the right tool when the entire subtree is provenance.
  • Not weakening the gate — forbidden tokens still rejected outside these four prefixes.
  • Not removing legacy names from archives — they remain audit evidence of the rebrand.

🤖 Generated with Claude Code

Four directory trees contain legacy brand names (TradePulse, HydroBrain,
neurotrade, NeuroPro) by design — they exist precisely to document the
old-repo → GeoSync rebrand and would lose audit value if scrubbed:

- .claude/archive/ — rebrand provenance ledgers
- tests/archive/   — tests that exercise the archive ledger schemas
- tools/archive/   — tooling that compares old-repo to new-repo by name

Added all three to SKIP_PREFIXES alongside the existing docs/archive entry.
Same pattern, same justification.

Also added .claude/worktrees/ — ephemeral Claude Code agent worktrees
that pollute the scan with copies of the same archive files.

Verified: 5291 files scanned, 91 allowlisted, 0 violations.
Brand-consistency self-tests: 24/24 pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@neuron7xLab neuron7xLab enabled auto-merge (squash) April 27, 2026 17:22
@neuron7xLab neuron7xLab merged commit 324c408 into main Apr 27, 2026
8 checks passed
@neuron7xLab neuron7xLab deleted the fix/brand-consistency-archive-allowlist branch April 27, 2026 17: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.

1 participant