Skip to content

refactor: resolve clawpatch findings#26

Merged
Jercik merged 10 commits into
mainfrom
fix/clawpatch-findings
Jun 14, 2026
Merged

refactor: resolve clawpatch findings#26
Jercik merged 10 commits into
mainfrom
fix/clawpatch-findings

Conversation

@Jercik

@Jercik Jercik commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • Consolidates duplicated WorktreeEntry and path-containment helpers.
  • Removes duplicated CLI metadata/type state and redundant OutputWriter option mirrors.
  • Deduplicates workflow configuration in commit-message and PR-review workflows.

Validation

  • pnpm format
  • pnpm typecheck
  • pnpm lint
  • pnpm test
  • pre-commit hook checks on each commit: format:check, knip, typecheck, lint, fta, test
  • ruby YAML parse for .github/workflows/commit-msg.yml and .github/workflows/pr-review.yml
  • clawpatch revalidate for all 9 findings

Clawpatch

Final clawpatch status --json reports openFindings: 0.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

@Jercik

This comment has been minimized.

@Jercik

This comment has been minimized.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

@Jercik Jercik merged commit 11223b5 into main Jun 14, 2026
5 checks passed
@Jercik Jercik deleted the fix/clawpatch-findings branch June 14, 2026 05:30
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