fix(ci): scope API client drift check to generated paths only

[FSM1]

Summary

• The CI drift check was diffing the entire packages/api-client/ directory, which would false-positive on hand-written changes to instance.ts, tests, or package.json
• Narrowed to only check openapi.json, src/generated/, and src/models/ — the actual generated outputs

Test plan

• CI passes

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Enhanced internal CI verification process for generated file consistency to ensure build integrity.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f282c4af-fac6-4b65-a267-aa93e2cbe2c7

📥 Commits

Reviewing files that changed from the base of the PR and between 374c880 and 742e967.

📒 Files selected for processing (1)

• .github/workflows/ci.yml

---

Walkthrough

Updated CI workflow step to broaden the scope of generated file verification, explicitly including packages/api-client/openapi.json, packages/api-client/src/generated/, and packages/api-client/src/models/ instead of checking the entire directory.

Changes

Cohort / File(s)	Summary
CI Workflow Configuration .github/workflows/ci.yml	Expanded generated file verification in the api-spec job's "Verify no uncommitted changes to generated files" step to explicitly check specific paths using a multi-line shell command passed to git diff --exit-code.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• refactor(web): consolidate API layer onto @cipherbox/api-client #309: Consolidation of API client into packages/api-client/ with associated changes to generated outputs and CI verification checks for these specific paths.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: scoping the CI drift check to only generated paths, which matches the core objective of narrowing the diff scope from the entire packages/api-client/ directory to only generated files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/ci-api-client-diff-scope

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.28%. Comparing base (374c880) to head (742e967).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #310      +/-   ##
==========================================
- Coverage   50.36%   50.28%   -0.08%     
==========================================
  Files         114      115       +1     
  Lines        9162     9854     +692     
  Branches      716      716              
==========================================
+ Hits         4614     4955     +341     
- Misses       4371     4722     +351     
  Partials      177      177              

Flag	Coverage Δ
api	82.76% <ø> (ø)
api-client	82.76% <ø> (ø)
core	82.76% <ø> (ø)
crypto	82.76% <ø> (ø)
desktop	28.64% <ø> (+2.73%)	⬆️
sdk	82.76% <ø> (ø)
sdk-core	82.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 10 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.