Skip to content

fix: update dependencies#22

Merged
angeloashmore merged 1 commit intomasterfrom
aa/update-infrastructure
Apr 4, 2026
Merged

fix: update dependencies#22
angeloashmore merged 1 commit intomasterfrom
aa/update-infrastructure

Conversation

@angeloashmore
Copy link
Copy Markdown
Member

@angeloashmore angeloashmore commented Apr 4, 2026

Resolves:

Description

Aligns this project's infrastructure with the current prismic-typescript-template to keep tooling and CI/CD consistent across Prismic packages.

Key changes:

  • Formatting: Replace Prettier with oxfmt (Rust-based, faster, simpler config)
  • Node engine: Bump minimum from >=14.15.0 to >=20
  • tsconfig: Switch to moduleResolution: "bundler", enable declaration/declarationMap
  • Linting: Add $schema to oxlintrc, remove padding-line-between-statements rule
  • CI/CD: Consolidate 4 separate publish workflows into a single publish.yml
  • .gitignore: Add *.tgz, remove stale playground/Next.js entries

Checklist

  • A comprehensive Linear ticket, providing sufficient context and details to facilitate the review of the PR, is linked to the PR.
  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA 1

  • Verify CI passes (lint, types, unit tests across Node 20/22/24, build)
  • Confirm npm run format uses oxfmt
  • Confirm publish workflow triggers correctly on push to master and on PRs

🤖 Generated with Claude Code


Note

Medium Risk
Medium risk because it changes the release/publish automation and bumps the minimum Node.js engine to >=20, which can affect consumers and CI behavior. Most other changes are tooling/config updates with minimal runtime impact.

Overview
Consolidates npm publishing automation by replacing the separate release, prerelease-canary, prerelease-pr, and PR cleanup workflows with a single publish.yml that handles releases (via release-please) plus canary/PR prereleases and tag cleanup.

Refreshes project tooling to match the template: replaces Prettier with oxfmt (adds .oxfmtrc.json, removes Prettier config/ignore), tweaks oxlint config (adds $schema, drops padding-line-between-statements), and updates .gitignore (ignore *.tgz, remove stale Next/playground entries).

Upgrades platform/deps by bumping engines.node to >=20, updating build/test tooling dependencies (TypeScript/Vitest/jsdom/tsdown/oxlint), and making small formatting-only changes in SimulatorAPI/SimulatorClient.

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

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

- Replace Prettier with oxfmt for formatting
- Update Node engine requirement from >=14.15.0 to >=20
- Update tsconfig: moduleResolution "bundler", enable declaration/declarationMap
- Update oxlintrc: add $schema, remove padding-line-between-statements
- Consolidate 4 publish workflows into single publish.yml
- Add MODE: production env to validate build job
- Clean up .gitignore (add *.tgz, remove stale entries)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 980651d. Configure here.

"oxfmt": "^0.43.0",
"oxlint": "^1.58.0",
"tsdown": "^0.21.7",
"typescript": "^6.0.2",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CI types matrix tests outdated TypeScript versions

High Severity

The typescript devDependency was updated to ^6.0.2 in package.json, but the validate.yml CI matrix for type-checking still only tests TypeScript 5.x versions. Since TypeScript 6.0 includes significant breaking changes, the project's actual TypeScript version isn't validated in CI, meaning type-checking regressions specific to TS 6.x might go undetected.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 980651d. Configure here.

@angeloashmore angeloashmore changed the title chore: update infrastructure to match prismic-typescript-template fix: update dependencies Apr 4, 2026
@angeloashmore angeloashmore merged commit a0bdcb9 into master Apr 4, 2026
13 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.

1 participant