Skip to content

fix: use -X as HTTP method flag for curl compatibility#22

Merged
reneaaron merged 2 commits intomasterfrom
fix/curl-compat-method-flag
Apr 9, 2026
Merged

fix: use -X as HTTP method flag for curl compatibility#22
reneaaron merged 2 commits intomasterfrom
fix/curl-compat-method-flag

Conversation

@reneaaron
Copy link
Copy Markdown
Member

@reneaaron reneaaron commented Apr 9, 2026

Summary

  • Changes fetch command's -m flag to -X to match curl convention
  • Every agent (and developer) expects -X POST — using -m caused agents to fail on first attempt
  • Bumps version to 0.6.1

Test plan

  • fetch -X POST works (tested with podcast search endpoint)
  • fetch --method POST long form works
  • Existing test suite passes (1 pre-existing flaky test unrelated)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Released version 0.6.1 of the CLI.
    • Updated the CLI option flag for specifying HTTP method (from -m to -X).

reneaaron and others added 2 commits April 9, 2026 23:32
Every agent and developer expects -X from curl. Using -m caused
agents to fail on their first attempt when constructing fetch commands.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Every agent and developer expects -X from curl. Using -m caused
agents to fail on their first attempt when constructing fetch commands.

Bumps version to 0.6.1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3cc4ee7d-ca9a-410c-b865-52078962df75

📥 Commits

Reviewing files that changed from the base of the PR and between d2088cd and b102ccb.

📒 Files selected for processing (3)
  • package.json
  • src/commands/fetch.ts
  • src/index.ts

📝 Walkthrough

Walkthrough

Patch version bump from 0.6.0 to 0.6.1 applied across package metadata and CLI entry point. HTTP method CLI option flag changed from -m to -X in the fetch command.

Changes

Cohort / File(s) Summary
Version Updates
package.json, src/index.ts
Version incremented from 0.6.0 to 0.6.1 across package manifest and CLI version declaration.
CLI Option Changes
src/commands/fetch.ts
HTTP method short flag updated from -m to -X while preserving long-form --method option and functionality.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • rolznz

Poem

🐰 A rabbit hops with glee so bright,
Version bumped to point-six-one, just right!
The fetch command wears a new flag today,
-X marks the way! Hop-hop, hooray! 🥕

✨ 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 fix/curl-compat-method-flag

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.

@reneaaron reneaaron merged commit 51d0156 into master Apr 9, 2026
1 of 2 checks passed
@reneaaron reneaaron deleted the fix/curl-compat-method-flag branch April 9, 2026 21:40
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