Skip to content

Accept canonical GitHub repository casing#1875

Merged
chubes4 merged 1 commit into
mainfrom
fix/1874-publication-repo-casing
Jul 18, 2026
Merged

Accept canonical GitHub repository casing#1875
chubes4 merged 1 commit into
mainfrom
fix/1874-publication-repo-casing

Conversation

@chubes4

@chubes4 chubes4 commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • compare authorized GitHub owner/repository identity case-insensitively during post-publication verification
  • retain strict HTTPS pull URL shape, pull number, exact API html_url, and target repository checks
  • cover canonical mixed-case acceptance and different-repository rejection

Fixes #1874.

How to test

  1. Run npm ci.
  2. Run npm run test:native-agent-task-lifecycle.
  3. Run npm run test:runner-workspace-publisher.
  4. Run git diff --check origin/main...HEAD.
  5. Confirm the lifecycle fixture accepts request target automattic/build-with-wordpress with canonical publication/API identity Automattic/build-with-wordpress.
  6. Confirm a publication for other/repo remains rejected.

All commands pass locally.

Compatibility

No extension-path or backwards-compatibility impact. GitHub repository identity is case-insensitive; URL syntax, pull identity, target binding, and API verification remain strict.

Source evidence

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode with OpenAI GPT-5.6 Sol
  • Used for: Diagnosed the post-publication casing mismatch, implemented fail-closed identity normalization, added regression coverage, and ran verification. Chris reviewed and owns the change.

@chubes4
chubes4 merged commit 1153e29 into main Jul 18, 2026
4 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.

Publication verification rejects canonical repository URL casing

1 participant