Skip to content

fix: drop calls/news commands for removed API endpoints; fix jobs columns - #13

Merged
jschfflr merged 1 commit into
mainfrom
fix/remove-calls-news-commands
Jul 10, 2026
Merged

fix: drop calls/news commands for removed API endpoints; fix jobs columns#13
jschfflr merged 1 commit into
mainfrom
fix/remove-calls-news-commands

Conversation

@jschfflr

Copy link
Copy Markdown
Contributor

Companion to apollo-api#20 (which removed list_contact_calls/list_account_news and rewired list_account_jobs).

Changes

  • Remove news listlist_account_news is gone (Apollo removed /accounts/{id}/news, no replacement).
  • Remove calls listlist_contact_calls is gone; calls search stays (uses the still-working search_calls).
  • jobs list — columns updated to the organization_job_postings shape the fixed list_account_jobs returns: Title / City / State / URL / Posted (the old Location/Department fields aren't in the new shape).

Testing

  • ruff ✅ · mypy ✅ · 81 tests ✅ · app loads with news gone, calls down to search, jobs present.
  • End-to-end verified against the fixed api: jobs list → 395 postings, calls search → 221, news command removed.

Notes

  • Depends on apollo-api#20; once that's released, bump the qodev-apollo-api floor here (or fold into a CLI release PR). CI passes today because the removed methods still exist in the pinned api build and nothing references them anymore.
  • No version bump (left for a release PR).

🤖 Generated with Claude Code

…umns

Apollo removed the nested sub-resource routes these relied on (see
apollo-api fix/remove-dead-list-endpoints):

- Remove 'news list' (list_account_news is gone; no replacement).
- Remove 'calls list' (list_contact_calls is gone); 'calls search' stays.
- 'jobs list' now surfaces the organization_job_postings shape returned by the
  fixed list_account_jobs: columns Title/City/State/URL/Posted (was
  Location/Department, which the new shape doesn't carry).

End-to-end verified against the fixed api: jobs list → 395 postings,
calls search → 221, news command removed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@peqy

peqy Bot commented Jul 10, 2026

Copy link
Copy Markdown

Looks good — clean removal of news/calls list and the jobs list columns update matches the new organization_job_postings shape.

[Improvement] Docs/changelog may now be stale

Fix: In a follow-up (or the release PR), remove/update any README/CHANGELOG references to the news command so users don’t see commands that no longer exist.

Found 1 issue (0 critical, 1 improvement).

@jschfflr
jschfflr merged commit 8159891 into main Jul 10, 2026
4 checks passed
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.

1 participant