Skip to content

[CLI] Add inference to cli query command#2395

Merged
nezaj merged 3 commits intomainfrom
cli-query-cardinality
Mar 20, 2026
Merged

[CLI] Add inference to cli query command#2395
nezaj merged 3 commits intomainfrom
cli-query-cardinality

Conversation

@nezaj
Copy link
Contributor

@nezaj nezaj commented Mar 20, 2026

What it says on the tin! Now instant-cli query should match what's in client sdk with inference

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f9916460-7259-4137-be09-f9da9b358bc8

📥 Commits

Reviewing files that changed from the base of the PR and between b229af1 and a167a41.

📒 Files selected for processing (3)
  • client/packages/cli/__tests__/e2e/cli.e2e.test.ts
  • client/packages/cli/src/index.js
  • client/packages/version/src/version.ts

📝 Walkthrough

Walkthrough

This PR adds a CLI e2e test validating one-to-many reverse relationship queries, updates the query handler to include an inference flag in POST requests, and bumps the version number from v0.22.165 to v0.22.166.

Changes

Cohort / File(s) Summary
E2E Test Addition
client/packages/cli/__tests__/e2e/cli.e2e.test.ts
Added new e2e test case asserting that querying through a has: "one" reverse relationship returns a single nested object rather than an array, using a comments-to-post one-to-many relationship schema.
Query Handler Update
client/packages/cli/src/index.js
Added inference? flag set to true in the POST request payload to /admin/query endpoint within the handleQuery function.
Version Bump
client/packages/version/src/version.ts
Updated exported version constant from 'v0.22.165' to 'v0.22.166'.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • dwwoelfel
  • stopachka
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title '[CLI] Add inference to cli query command' accurately summarizes the main change: adding an inference flag to the CLI query command.
Description check ✅ Passed The description 'Now instant-cli query should match what's in client sdk with inference' is directly related to the changeset, explaining the purpose of adding inference support.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cli-query-cardinality

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

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

Copy link
Contributor

@stopachka stopachka left a comment

Choose a reason for hiding this comment

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

Ship it!

@github-actions
Copy link
Contributor

View Vercel preview at instant-www-js-cli-query-cardinality-jsv.vercel.app.

@nezaj nezaj merged commit 27ce832 into main Mar 20, 2026
28 checks passed
@nezaj nezaj deleted the cli-query-cardinality branch March 20, 2026 20:34
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.

2 participants