fix: quote package test globs so Node expands them

[patoperpetua]

Summary

• Quote src/**/*.spec.ts (and .tsx in editor) in all six packages/*/package.json test scripts so Node's --test runner expands the glob instead of the shell.
• Matches the existing pattern in apps/api and prevents silently skipped specs when a package has tests at mixed directory depths.

Test plan

• pnpm test — all packages report the same test counts as before (email 95, client 9, compiler 12, types 4, editor 4, publisher 8, api 160)
• Verified nested spec pickup: temporary src/sub/throwaway-depth.spec.ts in post-kit-client ran with quoted glob, then removed
• pnpm lint and pnpm build pass locally

Closes #71

Made with Cursor

Summary by CodeRabbit

• Tests
  • Improved test command reliability across packages by preserving file patterns consistently across different shell environments.
  • Ensured all matching test files are passed correctly to the test runner.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f713e6e4-a222-4d12-8313-c1afebac1ba6

📥 Commits

Reviewing files that changed from the base of the PR and between 1e667ae and 3675a18.

📒 Files selected for processing (6)

• packages/post-kit-client/package.json
• packages/post-kit-compiler/package.json
• packages/post-kit-editor/package.json
• packages/post-kit-email/package.json
• packages/post-kit-publisher/package.json
• packages/post-kit-types/package.json

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.

---

📝 Walkthrough

Walkthrough

The package test scripts now quote their specification globs. Node’s test runner receives each glob without shell expansion.

Changes

Package test glob quoting

Layer / File(s)	Summary
Quote package test scripts packages/post-kit-*/package.json	The client, compiler, editor, email, publisher, and types test scripts now pass quoted globs to Node’s test runner. Other script behavior remains unchanged.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 3675a

This localized change makes nested package tests expand consistently without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the primary change: quoting package test globs so Node expands them.
Description check	✅ Passed	The description includes the linked issue, summary, rationale, test results, nested-spec verification, lint, and build checks. It omits the template's separate Feedback focus and Automated checks sect…
Linked Issues check	✅ Passed	The changes satisfy issue #71 by quoting the test glob in all six package scripts, preserving the existing script commands and runner configuration. The reported root tests, nested-spec verification, …
Out of Scope Changes check	✅ Passed	All changes are limited to the six package test scripts and directly support issue #71. No unrelated code, test content, runner, or configuration changes are reported.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…

Full details: Description check

Explanation

The description includes the linked issue, summary, rationale, test results, nested-spec verification, lint, and build checks. It omits the template's separate Feedback focus and Automated checks sections, but the required information is mostly present.

Full details: Linked Issues check

Explanation

The changes satisfy issue #71 by quoting the test glob in all six package scripts, preserving the existing script commands and runner configuration. The reported root tests, nested-spec verification, lint, and build checks support the acceptance criteria.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (6 skipped: 6 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/71-quote-test-globs

---

_{Comment @coderabbitai help to get the list of available commands.}

[patoperpetua]

@coderabbitai full review

[coderabbitai]

✅ Action performed

Full review finished.