fix(test): stabilise examples

[ryoppippi]

Stabilize example tests to run without network and prefer nullish operations.

Changes

• Examples: run with preload + .env; skip LLM-heavy files via SKIP_LLM_EXAMPLES.
• Test setup: load dotenv; use ??= defaults for OPENAI_API_KEY/STACKONE_API_KEY/SKIP_LLM_EXAMPLES.
• MSW mocks: add HRIS endpoints used by examples.
• RequestBuilder: replace null/undefined checks with == null / != null where appropriate.
• Snapshots: update to match current generated specs.

Results

• bun test: 135 pass, 0 fail.

---

Summary by cubic

Stabilizes example tests to run offline and reduces flakiness in CI. Also adopts nullish operations for env defaults and RequestBuilder checks, and updates snapshots to current specs.

• New Features

  • Run examples with Bun preload and .env to activate MSW and env loading.
  • Mock StackOne HRIS endpoints in MSW for deterministic example runs.
  • Skip LLM-heavy examples via SKIP_LLM_EXAMPLES (default 1).

• Refactors

  • Use dotenv and ??= for OPENAI_API_KEY, STACKONE_API_KEY, and SKIP_LLM_EXAMPLES in test setup.
  • Replace explicit null/undefined checks with == null / != null in RequestBuilder.
  • Refresh OpenAPI and toolset snapshots.
  • Result: 135 passing tests, 0 failing.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@90

commit: f17566c

[Copilot]

Pull Request Overview

This PR stabilizes example tests by adding MSW mocks for network dependencies and improving null/undefined checks with nullish coalescing operators.

Key changes:

• Enhanced test setup with environment variable defaults and LLM example skipping
• Added MSW mocks for StackOne HRIS endpoints to eliminate network dependencies
• Replaced explicit null/undefined checks with nullish coalescing in RequestBuilder

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
examples/examples.spec.ts	Added LLM example filtering and MSW preloading for isolated test runs
bun.test.setup.ts	Added dotenv loading and safe environment variable defaults using nullish assignment
mocks/handlers.ts	Added StackOne HRIS endpoint mocks for employee data
src/modules/requestBuilder.ts	Replaced explicit null/undefined checks with nullish equality operators
src/toolsets/tests/snapshots/stackone.spec.ts.snap	Updated snapshots with new date-time format and description changes
src/openapi/tests/snapshots/openapi-parser.spec.ts.snap	Updated snapshots with new date-time format and description changes

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[cubic-dev-ai]

1 issue found across 6 files

_{React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

[cubic-dev-ai]

Inconsistent field naming/type between list and detail responses ('phone_number' string vs 'phone_numbers' array). Align the property for consistency to avoid downstream confusion.

Prompt for AI agents

Address the following comment on mocks/handlers.ts at line 23:
<comment>Inconsistent field naming/type between list and detail responses (&#39;phone_number&#39; string vs &#39;phone_numbers&#39; array). Align the property for consistency to avoid downstream confusion.</comment>
<file context>
@@ -1,6 +1,30 @@
+ return HttpResponse.json({
+ id: params.id,
+ name: &#39;Michael Scott&#39;,
+ phone_numbers: [&#39;+1-555-0100&#39;],
+ });
+ }
</file context>

Suggested change

	phone_numbers: ['+1-555-0100'],
	phone_number: '+1-555-0100',

[glebedel]

LGTM