Skip to content

Conversation

@tokio-on-jupiter
Copy link
Contributor

Summary

  • Fix canary workflow npm registry configuration
  • All @jupiterone/* packages are published to npmjs.org, not GitHub packages
  • The previous configuration caused 404 errors for packages like @jupiterone/hierarchical-token-bucket

Test plan

@tokio-on-jupiter tokio-on-jupiter requested a review from a team as a code owner January 23, 2026 21:04
Copilot AI review requested due to automatic review settings January 23, 2026 21:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a misconfiguration in the canary workflow that was causing 404 errors when attempting to install JupiterOne packages. The workflow previously incorrectly configured npm to fetch @jupiterone/* packages from GitHub Packages registry, but all JupiterOne packages are actually published to npmjs.org.

Changes:

  • Removed GitHub Packages registry override for @jupiterone/* scope
  • Simplified npm authentication to use only npmjs.org registry

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Remove GitHub packages registry override (all @jupiterone/* packages are on npmjs.org)
- Remove redundant "Configure npm authentication" step (actions/setup-node handles this)
- The setup-node action with registry-url + NODE_AUTH_TOKEN is sufficient
@tokio-on-jupiter tokio-on-jupiter force-pushed the fix/canary-workflow-npm-registry branch from 16ed4d4 to 141ba47 Compare January 23, 2026 21:05
@tokio-on-jupiter tokio-on-jupiter enabled auto-merge (squash) January 23, 2026 21:08
@tokio-on-jupiter tokio-on-jupiter merged commit 07c7af0 into main Jan 23, 2026
8 checks passed
@tokio-on-jupiter tokio-on-jupiter deleted the fix/canary-workflow-npm-registry branch January 23, 2026 21:10
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.

3 participants