Skip to content

fix: add Claude workflow verification and modular rules#1399

Merged
KRRT7 merged 4 commits into
mainfrom
fix/claude-workflow-verification
Feb 6, 2026
Merged

fix: add Claude workflow verification and modular rules#1399
KRRT7 merged 4 commits into
mainfrom
fix/claude-workflow-verification

Conversation

@KRRT7

@KRRT7 KRRT7 commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add verification step to CI Claude workflow to prevent hallucinated fixes
  • Extract inline CLAUDE.md rules into modular .claude/rules/ files (architecture, code-style, git, source-code, testing)
  • Update architecture map with missing modules (languages/, setup/, picklepatch/, tracer.py, version.py)
  • Fix libcst rule to reflect actual practice: libcst for modification, ast acceptable for analysis

KRRT7 added 3 commits February 6, 2026 01:53
…ted fixes

The PR review bot was claiming lint issues were fixed without actually
fixing or committing them. Add a mandatory re-run of prek after fixes
and explicit instructions to report unfixed issues honestly.
Split monolithic CLAUDE.md instructions into focused, topic-specific
rule files. Path-scoped rules for source code and tests only load when
working with matching files.
@github-actions github-actions Bot added the workflow-modified This PR modifies GitHub Actions workflows label Feb 6, 2026
@KRRT7 KRRT7 force-pushed the fix/claude-workflow-verification branch from ba23b99 to 2847edc Compare February 6, 2026 07:30
@KRRT7 KRRT7 merged commit 3e4a2a8 into main Feb 6, 2026
25 of 28 checks passed
@KRRT7 KRRT7 deleted the fix/claude-workflow-verification branch February 6, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow-modified This PR modifies GitHub Actions workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant