Skip to content

fix: update string quotes in web search catalog - #845

Merged
thesujai merged 3 commits into
mainfrom
chore/fix-formatting
Sep 22, 2026
Merged

thesujai merged 3 commits into
mainfrom
chore/fix-formatting

Conversation

@thesujai

@thesujai thesujai commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #

Changes

How was this tested?

Checklist

  • I have read the contributing guidelines
  • pnpm build, pnpm test, pnpm typecheck, pnpm lint:ci, and pnpm format:check pass locally
  • Tests added/updated where it makes sense
  • No hand-edits to generated code (packages/trueforge-sdk, python/trueforge_sdk, .github/fern/openapi/openapi.json, docs/openapi.json) — fork PRs omit SDK regen; maintainers regenerate after merge
  • Docs / .env.example updated if configuration or behavior changed

Note

Low Risk
Build-time codegen only; same gitignore pattern as other catalog .gen.ts files with no runtime behavior change in source.

Overview
Treats webSearchCatalog.gen.ts like the other shipped catalog modules: it is added to .gitignore and the previously committed copy is removed from the repo.

The file is still produced by scripts/generate-catalog.mjs during build:gen from catalog/web-search-catalog.yaml, matching how model/MCP/skill/sandbox catalogs are handled.

Reviewed by Cursor Bugbot for commit 6b36ef6. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented Sep 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6b36ef6

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

@thesujai
thesujai enabled auto-merge (squash) September 22, 2026 13:06
@thesujai
thesujai merged commit 9ad5a7d into main Sep 22, 2026
10 checks passed
@thesujai
thesujai deleted the chore/fix-formatting branch September 22, 2026 14:00
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