Skip to content

fix(security): remove remaining regex redos paths#109

Closed
anand-testcompare wants to merge 4 commits intomainfrom
chore/require-codeql-in-check-gate
Closed

fix(security): remove remaining regex redos paths#109
anand-testcompare wants to merge 4 commits intomainfrom
chore/require-codeql-in-check-gate

Conversation

@anand-testcompare
Copy link
Collaborator

Summary

  • replace the remaining connector and slug regex paths in the Shopify source parser with linear parsing
  • add focused unit coverage for connector extraction and normalization

Validation

  • bun install --frozen-lockfile
  • bun test packages/shopify-cable-source/src/text.test.ts packages/shopify-cable-source/src/connector-text.test.ts
  • bun x ultracite fix
  • bun x ultracite check

@vercel
Copy link

vercel bot commented Mar 15, 2026

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

Project Deployment Actions Updated (UTC)
cable-intel-web Ready Ready Preview, Comment Mar 15, 2026 3:32am

Request Review

@github-actions github-actions bot enabled auto-merge (squash) March 15, 2026 03:30
@github-actions
Copy link
Contributor

Browser QA (agent-browser)

@anand-testcompare
Copy link
Collaborator Author

Superseded by #111, which merged the final CodeQL and CI fixes into main.

auto-merge was automatically disabled March 15, 2026 03:42

Pull request was closed

@anand-testcompare anand-testcompare deleted the chore/require-codeql-in-check-gate branch March 15, 2026 03:49
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.

1 participant