Skip to content

feat(seed): add short aliases for optional CLI params#14826

Merged
jsklan merged 1 commit into
mainfrom
seed-aliases
Apr 9, 2026
Merged

feat(seed): add short aliases for optional CLI params#14826
jsklan merged 1 commit into
mainfrom
seed-aliases

Conversation

@fern-support
Copy link
Copy Markdown
Collaborator

@fern-support fern-support commented Apr 9, 2026

Summary

  • Adds short aliases to all seed CLI optional params that were missing them
  • Single-word params get single-letter aliases (e.g. --fixture-f, --local-l)
  • Compound hyphenated params use initials matching word count (e.g. --skip-scripts-ss, --base-ref-br, --fern-repo-directory-frd)

Test plan

  • pnpm compile --filter @fern-api/seed-cli passes
  • Verify pnpm seed test --help shows aliases
  • Spot-check a few aliases work (e.g. pnpm seed test -g python-sdk -f basic-auth -ss)

Open with Devin

Add aliases to seed CLI options that were missing them, making
interactive usage faster. Compound params use initials matching
word count (e.g. --skip-scripts → -ss, --base-ref → -br,
--fern-repo-directory → -frd).
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

🌱 Seed Test Selector

Select languages to run seed tests for:

  • Python
  • TypeScript
  • Java
  • Go
  • Ruby
  • C#
  • PHP
  • Swift
  • Rust
  • OpenAPI

How to use: Click the ⋯ menu above → "Edit" → check the boxes you want → click "Update comment". Tests will run automatically and snapshots will be committed to this PR.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@jsklan jsklan enabled auto-merge (squash) April 9, 2026 14:23
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

SDK Generation Benchmark Results

Comparing PR branch against latest nightly baseline on main (2026-04-09T04:46:50Z).

Full benchmark table (click to expand)
Generator Spec main (generator) main (E2E) PR (generator) Delta
csharp-sdk square 95s 138s 96s +1s (+1.1%)
go-sdk square 107s 134s 106s -1s (-0.9%)
java-sdk square 290s 346s 291s +1s (+0.3%)
php-sdk square 85s 119s 83s -2s (-2.4%)
python-sdk square 130s 166s 128s -2s (-1.5%)
ruby-sdk-v2 square 115s 153s 120s +5s (+4.3%)
rust-sdk square 93s 95s 93s +0s (+0.0%)
swift-sdk square 104s 448s 106s +2s (+1.9%)
ts-sdk square 98s 134s 98s +0s (+0.0%)

main (generator): generator-only time via --skip-scripts (includes Docker image build, container startup, IR parsing, and code generation — this is the same Docker-based flow customers use via fern generate). main (E2E): full customer-observable time including build/test scripts (nightly baseline, informational). Delta is computed against generator-only baseline.
⚠️ = generation exited with a non-zero exit code (timing may not reflect a successful run).
Baseline from nightly runs on main (latest: 2026-04-09T04:46:50Z). Trigger benchmark-baseline to refresh.

@jsklan jsklan merged commit 47b314b into main Apr 9, 2026
269 checks passed
@jsklan jsklan deleted the seed-aliases branch April 9, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants