Skip to content

Conversation

@eluce2
Copy link
Collaborator

@eluce2 eluce2 commented Jan 7, 2026

Note

Introduces stricter quality gates and centralizes environment management while refining type handling across packages.

  • CI/CD: Split continuous-release into lint, typecheck, test (incl. 1Password + @proofkit/fmodata E2E), and build; release.yml adds manual trigger, TURBO env, and a quality job gating publish
  • Secrets/ENV: New root op.env; removes per-package op.env and updates scripts to reference ../../op.env
  • Tooling: Adds typecheck scripts across packages and a Turbo typecheck task; bumps turbo and pnpm; small lint script tweaks
  • CLI/Deploy: Safer handling of envFile (defaults to .env); fixes category typing; improves provider wrapping types
  • Typegen/Generators: Normalizes data source config (default type: "fmdapi"), narrows types to FmdapiDataSourceConfig, and filters ops by type
  • Registry: Replaces shadcn type imports with local lightweight schemas; updates registry package scripts
  • Auth installer: Ensures added config includes type: "fmdapi" and explicit envNames: undefined

Written by Cursor Bugbot for commit e0cc7d3. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Jan 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
proofkit-docs Ready Ready Preview Jan 10, 2026 7:58pm

@eluce2 eluce2 marked this pull request as ready for review January 7, 2026 16:37
Copy link
Collaborator Author

eluce2 commented Jan 7, 2026


release:
name: Release
needs: [quality]
Copy link

Choose a reason for hiding this comment

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

Missing npm authentication token breaks package publishing

High Severity

The NODE_AUTH_TOKEN environment variable was removed from the changesets/action step. Since actions/setup-node is configured with registry-url: "https://registry.npmjs.org", it creates an .npmrc that expects NODE_AUTH_TOKEN for authentication. Without this secret (previously set to ${{ secrets.NPM_ACCESS_TOKEN }}), the pnpm release command will fail when attempting to publish packages to npm with an authentication error.

Fix in Cursor Fix in Web

Copy link
Collaborator Author

eluce2 commented Jan 10, 2026

Merge activity

  • Jan 10, 7:57 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 10, 7:58 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 10, 7:59 PM UTC: @eluce2 merged this pull request with Graphite.

@eluce2 eluce2 changed the base branch from fix-typegen-default-api-key-auth to graphite-base/97 January 10, 2026 19:57
@eluce2 eluce2 changed the base branch from graphite-base/97 to main January 10, 2026 19:57
@eluce2 eluce2 force-pushed the 01-07-update_release_ci branch from 7a5b665 to e0cc7d3 Compare January 10, 2026 19:58
@eluce2 eluce2 merged commit 4803c68 into main Jan 10, 2026
8 of 10 checks passed
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