Skip to content

Use TypeScript for repository scripts and configuration - #393

Merged
admdly merged 3 commits into
mainfrom
chore/ts-updates
Oct 18, 2026
Merged

admdly merged 3 commits into
mainfrom
chore/ts-updates

Conversation

@admdly

@admdly admdly commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

The OpenAPI update script and Astro config still used JavaScript. Convert both to TypeScript, using Node.js 24's built-in support to run the update script without another dependency.

The update script now treats fetched JSON as unknown and narrows schema fields before inspecting them. Tests cover valid documents, malformed input, cursor and list-item validation, and HTTP failures. This branch also refreshes the generated Cloudflare worker types.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-10-18T01:18:11.516601Z 562ac3b PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://chore-ts-updates-extensions.fossbilling.workers.dev (commit 562ac3b)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://5fb41fde-extensions.fossbilling.workers.dev 562ac3b 2026-10-18T01:16:34.310Z Visit the dashboard ↗

@admdly admdly self-assigned this Oct 18, 2026

@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.

No issues found across 5 files

Auto-approved: Converts the OpenAPI update script and Astro config from JavaScript to TypeScript with runtime type narrowing, adds tests for the script, and refreshes generated Wrangler types. Dev-only tooling and test changes with no runtime behavior change.
Tip: Get a Slack DM when cubic finishes reviewing your PRs. Connect Slack

Turn on auto-fix | Re-trigger cubic

@admdly
admdly merged commit 82e3212 into main Oct 18, 2026
8 checks passed
@admdly
admdly deleted the chore/ts-updates branch October 18, 2026 01:18
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