Skip to content

Conversation

@barjin
Copy link
Member

@barjin barjin commented Nov 13, 2025

Fixes publishing of the docs-theme package by adding the repository field to its package.json (required for provenance generation).


Note

Add repository field to apify-docs-theme/package.json and update lockfile version to 1.0.223.

  • Package metadata:
    • Add repository block to apify-docs-theme/package.json.
  • Lockfile:
    • Bump @apify/docs-theme version in package-lock.json to 1.0.223.

Written by Cursor Bugbot for commit e4ecd7c. Configure here.

@barjin barjin requested a review from janbuchar November 13, 2025 10:27
@barjin barjin self-assigned this Nov 13, 2025
@barjin barjin added the adhoc Ad-hoc unplanned task added during the sprint. label Nov 13, 2025
@barjin barjin changed the title chore(ci): add repository entry to theme package.json chore(ci): add repository entry to theme package.json Nov 13, 2025
@github-actions github-actions bot added this to the 127th sprint - Tooling team milestone Nov 13, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Nov 13, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on November 24

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment @cursor review or bugbot run to trigger another review on this PR

"repository": {
"type": "git",
"url": "git@github.com:apify/apify-docs.git"
},
Copy link

Choose a reason for hiding this comment

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

Bug: Repository URL Mismatch Breaks Provenance

The repository.url uses SSH format (git@github.com:) instead of HTTPS format (git+https://github.com/). This is inconsistent with the root package.json which uses HTTPS format, and may prevent npm provenance generation from working correctly, which is the stated purpose of adding this field.

Fix in Cursor Fix in Web

Copy link
Contributor

Choose a reason for hiding this comment

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

He ain't lying, I guess we can play it safe @barjin?

Copy link
Member Author

Choose a reason for hiding this comment

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

npm will normalize it, but sure 👍

@apify-service-account
Copy link

Preview for this PR was built for commit e4ecd7c3 and is ready at https://pr-2087.preview.docs.apify.com!

Copy link
Contributor

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

LGTM

@apify-service-account
Copy link

Preview for this PR was built for commit 5279ca8e and is ready at https://pr-2087.preview.docs.apify.com!

@barjin barjin merged commit 6c24dd3 into master Nov 13, 2025
9 checks passed
@barjin barjin deleted the chore/oidc-theme-publishing branch November 13, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants