Skip to content

Add repository field to package.json (fixes E422 provenance on publish) - #5

Merged
MykhailoTamarin merged 1 commit into
masterfrom
tech/add-repository-field
Oct 1, 2026
Merged

MykhailoTamarin merged 1 commit into
masterfrom
tech/add-repository-field

Conversation

@MykhailoTamarin

@MykhailoTamarin MykhailoTamarin commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Why

The publish run to master failed with:

npm error 422 ... Error verifying sigstore provenance bundle: Failed to validate
repository information: package.json: "repository.url" is "", expected to match
"https://github.com/flipdishbytes/api-client-javascript" from provenance

npm OIDC provenance validation requires repository.url in package.json to match the GitHub repo. The generated package.json has no repository field (unlike @flipdish/api-client-typescript, which has one — same format used here).

What

  • Add "repository": {"type": "git", "url": "git+https://github.com/flipdishbytes/api-client-javascript.git"} to package.json.

Note

The generator (openapi-generator javascript template) does not emit this field — after the next Zeus regeneration it will disappear again unless the generator is fixed. Tracking that in the generator-repo work (Part 3).

Summary by CodeRabbit

  • Chores
    • Added repository details to the package metadata.

npm OIDC provenance validation requires package.json repository.url to match
the GitHub repo from the sigstore provenance bundle; the generated package.json
has no repository field, so publish failed with E422:
"package.json: repository.url is "", expected to match
https://github.com/flipdishbytes/api-client-javascript from provenance"
@MykhailoTamarin
MykhailoTamarin requested a review from a team as a code owner October 1, 2026 13:44
@coderabbitai

coderabbitai Bot commented Oct 1, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 0c0ab13a-bfe7-4af6-bb60-b092635941a7

📥 Commits

Reviewing files that changed from the base of the PR and between 8824685 and e3cf128.

📒 Files selected for processing (1)
  • package.json
 ________________________
< Tree-sitter is my GPS. >
 ------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@MykhailoTamarin
MykhailoTamarin merged commit 1889f79 into master Oct 1, 2026
2 of 4 checks passed
@MykhailoTamarin
MykhailoTamarin deleted the tech/add-repository-field branch October 1, 2026 13:45
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