Skip to content

ci: smoke-test framework adapter subpaths - #149

Merged
czaydev merged 1 commit into
czaydev:mainfrom
poyrazavsever:codex/adapter-subpath-smoke
Oct 8, 2026
Merged

czaydev merged 1 commit into
czaydev:mainfrom
poyrazavsever:codex/adapter-subpath-smoke

Conversation

@poyrazavsever

Copy link
Copy Markdown
Collaborator

What and why

Closes #148

The existing smoke tests resolve the core, testing and plugins entries, but do not check framework adapter exports. This adds package-name checks for next, express, hono, fastify, elysia and react-router in both CommonJS and ESM. Each check also verifies that the expected handler is a function.

The package-content step now requires every adapter's .js, .mjs and .d.ts artifacts. These checks use the existing Node 20/22/24 matrix. No runtime code, dependencies or public API changes.

Verification

  • Package lint, typecheck and build passed.
  • Unit/integration suite: 29 files, 464 tests passed.
  • Executed the new workflow smoke block locally: all six adapters passed in both module formats.
  • Executed the package-content block and npm pack --dry-run successfully.
  • Negative checks using modified in-memory workflow commands rejected an unexported subpath, a missing handler and a missing artifact. No source/package metadata was changed for these checks.
  • git diff --check and local vault/PR-safety checks passed.

Checklist

  • Lint, typecheck and tests pass
  • Positive and negative smoke checks cover the change
  • No changeset needed (CI-only)
  • No EN/TR docs or Since badge needed (no user-facing change)
  • No credentials, card data or personal vault files

@poyrazavsever
poyrazavsever requested a review from czaydev as a code owner October 8, 2026 06:22
Copilot AI balanced review requested due to automatic review settings October 8, 2026 06:22
@vercel

vercel Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

@poyrazavsever is attempting to deploy a commit to the CzayLabs Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@czaydev
czaydev merged commit c4b0ed1 into czaydev:main Oct 8, 2026
5 of 6 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.

CI: smoke-test framework adapter subpath exports

3 participants