Skip to content

chore: Sync CLI with list quotes endpoint removal#323

Open
claude[bot] wants to merge 1 commit intomainfrom
docs/sync-20260409
Open

chore: Sync CLI with list quotes endpoint removal#323
claude[bot] wants to merge 1 commit intomainfrom
docs/sync-20260409

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Apr 9, 2026

Summary

  • Removed quotes list CLI command since GET /quotes endpoint was removed in remove list quotes endpoint #321
  • Cleaned up unused PaginatedResponse import in quotes.ts

Additional updates needed

The following files in .claude/skills/ also reference the removed endpoint but require manual updates due to permission restrictions:

  1. .claude/skills/grid-api/references/endpoints.md (line 77)

    • Remove: | GET | /quotes | List quotes (paginated) |
  2. .claude/skills/grid-api/SKILL.md (lines 327-329)

    • Remove the "List quotes" curl example

Test plan

🤖 Generated with Claude Code

The GET /quotes list endpoint was removed in PR #321.
This removes the corresponding CLI command and unused import.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
grid-flow-builder Ready Ready Preview, Comment Apr 9, 2026 8:40am

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 9, 2026

Greptile Summary

Removes the quotes list CLI command and its associated PaginatedResponse import from cli/src/commands/quotes.ts, syncing the CLI with the GET /quotes endpoint removal from PR #321. The remaining get, create, and execute subcommands are untouched and correct.

Confidence Score: 5/5

Safe to merge — straightforward removal of a defunct endpoint with no side effects.

Single-file change removes one subcommand and one unused import; remaining commands are untouched and the change is fully aligned with the OpenAPI spec update in #321.

No files require special attention. The PR description notes two skill files (.claude/skills/) still reference the removed endpoint, but those are out of scope for this PR.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
cli/src/commands/quotes.ts Removed quotes list subcommand and unused PaginatedResponse import; remaining commands (get, create, execute) are unaffected and correct.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[quotes command] --> B[get quoteId]
    A --> C[create]
    A --> D[execute quoteId]
    A -.->|removed| E[list]

    style E stroke-dasharray: 5 5,color:#999
Loading

Reviews (1): Last reviewed commit: "chore: Remove quotes list command (endpo..." | Re-trigger Greptile

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.

0 participants