Skip to content

Conversation

@monadoid
Copy link
Contributor

@monadoid monadoid commented Dec 24, 2025

why

what changed

test plan


Summary by cubic

Bumped server package to 3.1.3 and fixed the gen:openapi script so OpenAPI generation works on Windows. The script now uses && and formats the local openapi.v3.yaml without changing directories.

Written for commit 1b850c8. Summary will update automatically on new commits.

@changeset-bot
Copy link

changeset-bot bot commented Dec 24, 2025

⚠️ No Changeset found

Latest commit: 1b850c8

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

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 24, 2025

Greptile Summary

Version bump to 3.1.3 and Windows compatibility fix for the gen:openapi script. The script now uses && instead of ; (which is bash-specific) and simplifies the path resolution by staying in the current directory instead of navigating to the repo root.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Simple, well-scoped changes: a standard version bump and a cross-platform compatibility fix for a build script. The Windows fix correctly replaces bash-specific syntax with cross-platform compatible alternatives while maintaining the same functionality
  • No files require special attention

Important Files Changed

Filename Overview
packages/server/package.json Version bumped to 3.1.3; gen:openapi script fixed for Windows compatibility by replacing semicolon with && and simplifying path resolution

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant NPM as npm/pnpm
    participant TSX as tsx Runtime
    participant Script as gen-openapi.ts
    participant FS as File System
    participant Prettier as Prettier

    Dev->>NPM: pnpm run gen:openapi
    NPM->>TSX: tsx scripts/gen-openapi.ts
    TSX->>Script: Execute script
    Script->>Script: Initialize Fastify with OpenAPI
    Script->>Script: Register routes & schemas
    Script->>Script: Generate OpenAPI YAML
    Script->>Script: Post-process YAML
    Script->>FS: Write to openapi.v3.yaml
    FS-->>Script: File written
    Script-->>TSX: Exit successfully
    TSX-->>NPM: Command 1 complete
    NPM->>Prettier: prettier --write openapi.v3.yaml
    Prettier->>FS: Read openapi.v3.yaml
    FS-->>Prettier: File content
    Prettier->>Prettier: Format YAML
    Prettier->>FS: Write formatted content
    FS-->>Prettier: File updated
    Prettier-->>NPM: Formatting complete
    NPM-->>Dev: gen:openapi complete
Loading

@monadoid monadoid merged commit 6e44194 into main Dec 24, 2025
21 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.

3 participants