Skip to content

Fix committed worker scope checks#7

Merged
SetraTheXX merged 1 commit into
mainfrom
fix/committed-worker-scope-check
May 30, 2026
Merged

Fix committed worker scope checks#7
SetraTheXX merged 1 commit into
mainfrom
fix/committed-worker-scope-check

Conversation

@SetraTheXX

Copy link
Copy Markdown
Owner

Summary

Fixes worker scope validation so committed branch changes are checked in addition to git status --short.

Fixed

  • Adds baseCommit to registered worktrees
  • Checks git diff --name-only <baseCommit>...HEAD
  • Prevents workers from committing files outside allowedFiles
  • Prevents committed forbidden file changes from passing post-check
  • Keeps .cewp-worker-output/** as allowed runtime output

Safety

  • Old runs without baseCommit fail safely
  • No merge/push/publish automation added
  • No package version bump

Verification

  • committed allowed change PASS
  • committed outside allowed FAIL
  • committed forbidden FAIL
  • mixed committed/uncommitted scope test PASS
  • npm pack --dry-run PASS

Copilot AI review requested due to automatic review settings May 30, 2026 21:11
@SetraTheXX SetraTheXX merged commit 8262441 into main May 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@SetraTheXX SetraTheXX deleted the fix/committed-worker-scope-check branch May 30, 2026 21:39
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