Skip to content

chore: strip label-convention enforcement (schema unification prep)#49

Merged
samtuckerdavis merged 4 commits intomainfrom
chore/strip-label-conventions-2026-04-28
Apr 27, 2026
Merged

chore: strip label-convention enforcement (schema unification prep)#49
samtuckerdavis merged 4 commits intomainfrom
chore/strip-label-conventions-2026-04-28

Conversation

@OriginalGary
Copy link
Copy Markdown
Contributor

@OriginalGary OriginalGary commented Apr 27, 2026

Summary

Strips label-convention enforcement from the upstream rules + workflow that propagates across the org as prep for unifying the label schema from a clean slate.

Changes

  • Delete .claude/rules/pipeline-nevers.md (encodes label-driven gate semantics)
  • Delete .claude/rules/parallelization.md (encodes label-driven wave dispatch)
  • Delete .claude/rules/desloppify.md (references override labels)
  • Delete .github/workflows/setup-labels.yml (creates the labels we're tearing down)

Out of scope (intentionally)

  • claude-code/.claude/skills/{issue,merge,run,unblock}/SKILL.md — these are layer-3 dispatcher skills which the operator controls and has stopped. Left for separate handling alongside the new schema rebuild.

Why

Org-wide label state is fragmented across 30 repos. Layer-3 dispatcher is stopped, so this won't break in-flight automation. After merge, the automated-sync workflow will propagate these deletions to the 10 downstream linter-rule repos.

Companion PRs

  • Open-Paws/context#120 — strips labels.yaml + sync-labels.sh from the canonical schema repo
  • (forthcoming) Open-Paws/.github — removes the reusable auto-merge.yml workflow + label-template files
  • (forthcoming) per-downstream-repo: removes .github/workflows/auto-merge.yml callers

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed internal development configuration files including orchestration rules, parallelization guidelines, and pipeline constraints documentation.
    • Removed automated CI/CD workflow for repository label initialization.

@OriginalGary OriginalGary self-assigned this Apr 27, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9311e03b-abca-49e8-885d-b247465e25c2

📥 Commits

Reviewing files that changed from the base of the PR and between c187276 and f8e45d4.

📒 Files selected for processing (4)
  • .claude/rules/desloppify.md
  • .claude/rules/parallelization.md
  • .claude/rules/pipeline-nevers.md
  • .github/workflows/setup-labels.yml
💤 Files with no reviewable changes (4)
  • .github/workflows/setup-labels.yml
  • .claude/rules/desloppify.md
  • .claude/rules/pipeline-nevers.md
  • .claude/rules/parallelization.md

📝 Walkthrough

Walkthrough

This change removes 238 lines across four files: three Claude instruction rule documents (desloppify, parallelization, pipeline-nevers policies) and one GitHub Actions workflow that provisioned repository labels. No code logic is altered—purely documentation and infrastructure deletions.

Changes

Cohort / File(s) Summary
Claude Instruction Rules
.claude/rules/desloppify.md, .claude/rules/parallelization.md, .claude/rules/pipeline-nevers.md
Removes three rule specification documents totaling 222 lines: desloppify enforcement (Open-Paws fork, score gates, verification flow), parallelization constraints (worktree isolation, wave orchestration, TTL labeling, bootstrap fallback), and pipeline hard constraints (no skips, RED→GREEN enforcement, test isolation, merge discipline, conventional commits).
GitHub Workflow
.github/workflows/setup-labels.yml
Deletes the manual trigger workflow that created three labels (auto-merge, needs-human-review, human-gate) using gh CLI with error suppression.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the primary campaign: removing label-convention enforcement artifacts in preparation for unified schema migration.
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.
No Speciesist Idioms ✅ Passed Pull request contains no speciesist idioms. 'Kill process' is standard technical terminology for computer process management, not speciesist language.
No Hardcoded Secrets Or Credentials ✅ Passed Git history investigation of four deleted files found no hardcoded credentials. Only legitimate GitHub Actions secret reference pattern detected.
No Tier 3 Data Committed ✅ Passed PR deletes only procedural configuration files with no Tier 3 sensitive data—activist names, legal strategy, or NGO financials—present or committed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/strip-label-conventions-2026-04-28
  • 🛠️ fix NAV violations: Commit on current branch
  • 🛠️ fix NAV violations: Create PR

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

@samtuckerdavis samtuckerdavis merged commit 9c937a3 into main Apr 27, 2026
6 checks passed
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