Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@gcarvelli
Copy link
Contributor

Description

I believe these outdated URLs are the cause of our NPM publishing failures. Trusted Publisher expects this URL to match the URL of the repo.

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

@gcarvelli gcarvelli self-assigned this Dec 30, 2025
@gcarvelli gcarvelli requested a review from a team as a code owner December 30, 2025 18:56
@gcarvelli gcarvelli requested review from dandorman and hexedpackets and removed request for dandorman December 30, 2025 18:56
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 30, 2025

Greptile Summary

Updated three repository URLs in package.json from workos-inc to workos organization to fix NPM Trusted Publisher validation failures.

  • Changed homepage from https://github.com/workos-inc/workos-node to https://github.com/workos/workos-node
  • Changed repository.url from https://github.com/workos-inc/workos-node to git+https://github.com/workos/workos-node.git (now follows NPM standard format with git+ prefix and .git suffix)
  • Changed bugs.url from https://github.com/workos-inc/workos-node/issues to https://github.com/workos/workos-node/issues

These changes align the package metadata with the actual GitHub repository location and satisfy NPM Trusted Publisher's requirement that the repository URL must match the actual repository.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The changes are purely metadata updates to correct repository URLs in package.json. No code logic, dependencies, or functionality is affected. The URL corrections align with NPM best practices and fix the Trusted Publisher validation issue.
  • No files require special attention

Important Files Changed

Filename Overview
package.json Updated repository URLs from workos-inc to workos organization, fixing NPM Trusted Publisher validation

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Repository
    participant NPM as NPM Registry
    participant TP as Trusted Publisher

    Dev->>GH: Push URL fix to package.json
    Note over GH: URLs updated:<br/>workos-inc → workos<br/>Add git+ prefix & .git suffix
    Dev->>GH: Create PR #1428
    GH->>TP: Validate repository URL
    Note over TP: Check package.json repository.url<br/>matches GitHub repo URL
    TP-->>NPM: URL validation passes
    NPM->>NPM: Allow package publish
    Note over NPM: @workos-inc/node published successfully
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@gcarvelli gcarvelli merged commit 4cc6028 into main Dec 30, 2025
7 checks passed
@gcarvelli gcarvelli deleted the gio/attempt-oidc-fix branch December 30, 2025 19:05
@gcarvelli gcarvelli mentioned this pull request Dec 30, 2025
gcarvelli added a commit that referenced this pull request Dec 30, 2025
## Description

This version contains:
- #1428

## Documentation

Does this require changes to the WorkOS Docs? E.g. the [API
Reference](https://workos.com/docs/reference) or code snippets need
updates.

```
[ ] Yes
```

If yes, link a related docs PR and add a docs maintainer as a reviewer.
Their approval is required.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants