Skip to content

Release workos-authkit from CI with Changesets - #84

Merged
ianmacartney merged 2 commits into
mainfrom
ian/release-from-ci
Nov 11, 2026
Merged

ianmacartney merged 2 commits into
mainfrom
ian/release-from-ci

Conversation

@ianmacartney

Copy link
Copy Markdown
Member

Adopts the shared staged npm publishing and Changesets workflows. check stays offline and fast; CI initializes a local anonymous deployment and sets placeholder WorkOS values before the separate E2E/codegen gate, without CONVEX_AGENT_MODE or real credentials. Adds .convex/ to Prettier and Git ignores. Local build, check, anonymous E2E, generated-code comparison, and workflow lint passed. No package version bump or publication is included.

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@pkg-pr-new

pkg-pr-new Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@convex-dev/workos-authkit@84

commit: fda0696

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

1 issue found across 12 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name=".github/workflows/test.yml">

<violation number="1" location=".github/workflows/test.yml:57">
P2: `pkg-pr-new` cannot authenticate this job with only `contents: read`, and the fallback reports success when publishing fails. Grant `id-token: write` and fail the step on publication errors.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

git diff --exit-code -- src/component/_generated example/convex/_generated
test -z "$(git ls-files --others --exclude-standard src/component/_generated example/convex/_generated)"
- name: Publish package for testing branch
run: npx pkg-pr-new publish || echo "pkg-pr-new failed"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2: pkg-pr-new cannot authenticate this job with only contents: read, and the fallback reports success when publishing fails. Grant id-token: write and fail the step on publication errors.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. At .github/workflows/test.yml, line 57:

<comment>`pkg-pr-new` cannot authenticate this job with only `contents: read`, and the fallback reports success when publishing fails. Grant `id-token: write` and fail the step on publication errors.</comment>

<file context>
@@ -4,31 +4,54 @@ concurrency:
+          git diff --exit-code -- src/component/_generated example/convex/_generated
+          test -z "$(git ls-files --others --exclude-standard src/component/_generated example/convex/_generated)"
+      - name: Publish package for testing branch
+        run: npx pkg-pr-new publish || echo "pkg-pr-new failed"
</file context>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's fine for this to fail. and I don't think it needs that permission. if we see issues we can fix forward

@ianmacartney
ianmacartney merged commit aa73c60 into main Nov 11, 2026
6 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