Skip to content

fix: repository.url for npm provenance validation - #45

Merged
ryandmonk merged 1 commit into
mainfrom
fix/repository-field
Aug 11, 2026
Merged

fix: repository.url for npm provenance validation#45
ryandmonk merged 1 commit into
mainfrom
fix/repository-field

Conversation

@ryandmonk

Copy link
Copy Markdown
Contributor

Trusted publishing auth now works; provenance verification requires repository.url matching the workflow repo, and emit's package.json had none (all prior releases were manual, so it never surfaced). Mirrors dspack-gen. 312/312 green.

🤖 Generated with Claude Code

Trusted publishing now authenticates (E404 gone) but provenance verification
rejects the tarball: package.json carried no repository field, and sigstore
validation requires it to match the workflow's repo. Mirrors dspack-gen's shape.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 11, 2026 01:30
@ryandmonk
ryandmonk merged commit 3b152ae into main Aug 11, 2026
2 checks passed
@ryandmonk
ryandmonk deleted the fix/repository-field branch August 11, 2026 01:31

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.

Pull request overview

Adds repository metadata to @aestheticfunction/dspack-emit so npm trusted publishing provenance validation can associate the published artifact with the GitHub repository used by the release workflow.

Changes:

  • Add repository (type + url) to package.json to satisfy npm provenance verification requirements.
  • Minor formatting/indent normalization on the adjacent description line.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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