Skip to content

Comments

feat(cli auth): allow passing of token for cli commands. dep updates#1821

Open
cmwhited wants to merge 2 commits intomainfrom
chris.whited/amp-ts/feat-support-auth-token-in-cmds
Open

feat(cli auth): allow passing of token for cli commands. dep updates#1821
cmwhited wants to merge 2 commits intomainfrom
chris.whited/amp-ts/feat-support-auth-token-in-cmds

Conversation

@cmwhited
Copy link
Contributor

Description

  • bump ts deps
  • for the cli commands to interact with datasets, allow for passing in a token (or set as AMP_AUTH_TOKEN env var)

This fix is to resolve an issue that came up from the staging and prod amp instances now block usage without a JWT that we generate and distribute; this lets us pass that token in.


export const AUTH_PLATFORM_URL = new URL("https://auth.amp.thegraph.com/")

const AMP_JWKS = jose.createLocalJWKSet({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we only have the staging env of the registry currently, so this is the necessary JWKS

@cmwhited cmwhited requested a review from Theodus February 19, 2026 20:53
@LNSD
Copy link
Contributor

LNSD commented Feb 20, 2026

@cmwhited https://github.com/edgeandnode/amp/actions/runs/22200005362/job/64212147669?pr=1821

 Error: No projects were found. Make sure your configuration is correct. The projects definition: [
        "typescript/*/vitest.config.ts"
    ].

@cmwhited
Copy link
Contributor Author

@cmwhited https://github.com/edgeandnode/amp/actions/runs/22200005362/job/64212147669?pr=1821

 Error: No projects were found. Make sure your configuration is correct. The projects definition: [
        "typescript/*/vitest.config.ts"
    ].

@LNSD those tests have now failed 3 time in 3 separate runs with 3 separate errors. so not sure what the issue is

@cmwhited cmwhited force-pushed the chris.whited/amp-ts/feat-support-auth-token-in-cmds branch from ace2489 to c8ddaca Compare February 20, 2026 19:11
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