feat: snapi by jacekradko · Pull Request #6062 · clerk/javascript

jacekradko · 2025-06-04T14:45:15Z

Description

Fixes: SDKI-954

Checklist

pnpm test runs as expected.
pnpm build runs as expected.
(If applicable) JSDoc comments have been added or updated for any package exports
(If applicable) Documentation has been updated

Type of change

changeset-bot · 2025-06-04T14:45:19Z

⚠️ No Changeset found

Latest commit: 8614418

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel · 2025-06-04T14:45:46Z

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

Project	Deployment	Preview	Comments	Updated (UTC)
clerk-js-sandbox	Ready	Preview	Comment	Sep 23, 2025 6:53pm

+
+  async checkoutBranch(branch: string): Promise<void> {
+    try {
+      execSync(`git checkout ${branch}`, {


+
+  async getLastCommit(branch: string): Promise<string> {
+    try {
+      const commit = execSync(`git rev-parse ${branch}`, {


To fix the issue, we will replace the use of execSync with string interpolation by using a safer API, execFileSync, which allows passing arguments as an array. This approach avoids shell interpretation of special characters in the input. Specifically:

Replace the execSync call on line 34 with execFileSync, passing branch as an argument in an array.

Ensure that the branch parameter is passed as-is without being interpolated into a shell command string.

No additional dependencies are required for this fix.

+
+  async fetchBranch(branch: string): Promise<void> {
+    try {
+      execSync(`git fetch origin ${branch}`, {


To fix the issue, we will replace the use of execSync with a safer alternative. Specifically, we will use child_process.execFileSync, which allows us to pass arguments as an array, avoiding the need for shell interpretation. This approach ensures that special characters in the branch parameter are treated as literal strings rather than being interpreted by the shell.

The changes will involve:

Replacing the string interpolation in the execSync call with an execFileSync call.

Passing the branch parameter as an argument in an array to execFileSync.

github-actions · 2025-06-04T16:09:32Z

🔍 API Changes Report

✅ No API Changes

Summary

✅ No API changes detected in any packages.

All public APIs remain stable and backward compatible.

Next Steps

✅ No API changes detected.

Safe to merge.

This report was generated by the API Breakage Detector. For more information, see the documentation.

_{🤖 This comment was automatically generated by the API Breakage Detector}

clerk-cookie · 2025-07-27T00:01:38Z

Hello 👋

We currently close PRs after 60 days of inactivity. It's been 50 days since the last update here. If we missed this PR, please reply here. Otherwise, we'll close this PR in 10 days.

Thanks for being a part of the Clerk community! 🙏

clerk-cookie · 2025-09-18T00:01:08Z

Hello 👋

We currently close PRs after 60 days of inactivity. It's been 50 days since the last update here. If we missed this PR, please reply here. Otherwise, we'll close this PR in 10 days.

Thanks for being a part of the Clerk community! 🙏

- Combined both sets of tasks in turbo.json (machine/custom integration tests and api snapshot tasks) - Regenerated pnpm-lock.yaml to resolve dependency conflicts

coderabbitai · 2025-09-23T18:49:38Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch jr.feat/snapi

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

clerk-cookie · 2025-11-13T00:01:17Z

Hello 👋

We currently close PRs after 60 days of inactivity. It's been 50 days since the last update here. If we missed this PR, please reply here. Otherwise, we'll close this PR in 10 days.

Thanks for being a part of the Clerk community! 🙏

jacekradko added the do not merge label Jun 4, 2025

jacekradko changed the title ~~feat:brkr-brkr~~ feat: snapi Jun 4, 2025

jacekradko marked this pull request as draft June 4, 2025 14:45

clerk-cookie added the actions label Jun 4, 2025

feat: snapi

137968c

jacekradko force-pushed the jr.feat/snapi branch from 5d7bbd3 to 137968c Compare June 4, 2025 14:47

vercel Bot deployed to Preview June 4, 2025 14:49

View deployment

wip

e213be3

vercel Bot deployed to Preview June 4, 2025 15:13

View deployment

wip

bf41bd9

vercel Bot deployed to Preview June 4, 2025 15:16

View deployment

wip

ffdec4d

vercel Bot deployed to Preview June 4, 2025 15:20

View deployment

wip

6a7f351

vercel Bot deployed to Preview June 4, 2025 15:25

View deployment

use blacksmith

8ffd044

vercel Bot deployed to Preview June 4, 2025 15:33

View deployment

rename job

600c84b

vercel Bot deployed to Preview June 4, 2025 15:34

View deployment

wip

d5df082

vercel Bot deployed to Preview June 4, 2025 15:39

View deployment

github-advanced-security AI found potential problems Jun 4, 2025

View reviewed changes

wip

2de0222

vercel Bot deployed to Preview June 4, 2025 16:03

View deployment

wip

f63c0ec

vercel Bot deployed to Preview June 4, 2025 16:08

View deployment

wip

ccd8a03

narrow snapshoting

409e592

vercel Bot deployed to Preview June 4, 2025 16:35

View deployment

wip

51df0d3

vercel Bot deployed to Preview June 4, 2025 17:51

View deployment

github-advanced-security AI found potential problems Jun 4, 2025

View reviewed changes

Comment thread tools/snapi/src/utils/api-extractor.ts Fixed

Comment thread tools/snapi/src/utils/api-extractor.ts Fixed

Comment thread tools/snapi/src/utils/api-extractor.ts Fixed

wip

86404b8

vercel Bot deployed to Preview June 4, 2025 18:05

View deployment

wip

5020824

vercel Bot deployed to Preview June 4, 2025 18:50

View deployment

wip

5ddb59d

vercel Bot deployed to Preview June 4, 2025 18:56

View deployment

wip

67c0cef

clerk-cookie added the backend label Jun 6, 2025

vercel Bot deployed to Preview June 6, 2025 17:35

View deployment

clerk-cookie added the Stale label Jul 27, 2025

Merge branch 'main' into jr.feat/snapi

65a216e

vercel Bot deployed to Preview July 28, 2025 16:14

View deployment

clerk-cookie removed the Stale label Jul 29, 2025

clerk-cookie added the Stale label Sep 18, 2025

Resolve merge conflicts in turbo.json and pnpm-lock.yaml

8614418

- Combined both sets of tasks in turbo.json (machine/custom integration tests and api snapshot tasks) - Regenerated pnpm-lock.yaml to resolve dependency conflicts

vercel Bot deployed to Preview September 23, 2025 18:53

View deployment

clerk-cookie removed the Stale label Sep 24, 2025

clerk-cookie added Stale and removed Stale labels Nov 13, 2025

jacekradko closed this Jan 24, 2026

jacekradko deleted the jr.feat/snapi branch January 24, 2026 02:25

@@ -33,3 +33,3 @@
                 try {
-                  const commit = execSync(`git rev-parse ${branch}`, {
+                  const commit = execSync('git', ['rev-parse', branch], {
                     cwd: this.workspaceRoot,

@@ -1,2 +1,2 @@
-            import { execSync } from 'child_process';
+            import { execSync, execFileSync } from 'child_process';
@@ -91,3 +91,3 @@
                 try {
-                  execSync(`git fetch origin ${branch}`, {
+                  execFileSync('git', ['fetch', 'origin', branch], {
                     cwd: this.workspaceRoot,

Conversation

jacekradko commented Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Type of change

Uh oh!

changeset-bot Bot commented Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ No Changeset found

Uh oh!

vercel Bot commented Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Check warning

Uh oh!

Uh oh!

Check warning

Uh oh!

Uh oh!

Copilot Autofix

Check warning

Uh oh!

Uh oh!

Copilot Autofix

github-actions Bot commented Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔍 API Changes Report

✅ No API Changes

Summary

Next Steps

Uh oh!

Uh oh!

Uh oh!

Uh oh!

clerk-cookie commented Jul 27, 2025

Uh oh!

clerk-cookie commented Sep 18, 2025

Uh oh!

coderabbitai Bot commented Sep 23, 2025

Review skipped

Uh oh!

clerk-cookie commented Nov 13, 2025

Uh oh!

This branch was successfully deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jacekradko commented Jun 4, 2025 •

edited

Loading

changeset-bot Bot commented Jun 4, 2025 •

edited

Loading

vercel Bot commented Jun 4, 2025 •

edited

Loading

github-actions Bot commented Jun 4, 2025 •

edited

Loading