Skip to content

docs: comprehensive CLI command reference (closes #65)#134

Merged
rylinjames merged 1 commit into
mainfrom
docs/cli-command-reference
May 15, 2026
Merged

docs: comprehensive CLI command reference (closes #65)#134
rylinjames merged 1 commit into
mainfrom
docs/cli-command-reference

Conversation

@rylinjames
Copy link
Copy Markdown
Collaborator

Summary

Adds docs/cli_reference.md — a single user-facing reference for every visible reflex command. Closes #65.

Adapted from #127

@DsThakurRawat opened #127 with the same goal. The doc structure (verb overview → per-verb sections → vertical matrix → env vars) is theirs and good; preserved via Co-Authored-By on the commit.

What this PR fixes vs #127

#127 issue Fix here
Features reflex turbo as a current verb Dropped — source says it's deprecated and prints a deprecation banner pointing to reflex serve --adaptive-steps
Features reflex bench as user-facing Dropped — hidden=True + internal-only per v0.9.5 CHANGELOG
Features reflex validate-dataset and other hidden verbs as top-level Dropped — hidden=True in source
reflex ros2-serve treated as a normal top-level verb Kept, but moved to "Advanced commands" section with a "hidden from --help but production-supported" caveat; also adds the new --state-msg-type flag from #133
Claimed "AGV / GPS-fused state" guidance as a Reflex feature Removed — not a feature Reflex ships
Vertical matrix had retail / traffic / etc. dropped during my first review pass Restored — verified against 02_research/customers/MASTER.md in the FastCrest research vault: traffic-management, retail, warehouse-camera, drone surveillance are all P0 customer subcategories (12-13/15 composite scores). The matrix now mirrors the vault's top 11 P0 picks.
Flag tables had invented entries Trimmed to verified key flags only, with reflex <verb> --help pointers for the full list (prevents future drift)

What I added beyond fixing

  • Documents --state-msg-type {joint_state|imu|odom} introduced in fix(ros2): pluggable state extractors (joint_state / imu / odom) #133, with the right MAVROS topic recommendations for drones.
  • "Advanced commands" section listing ros2-serve and replay (hidden from --help but production-supported) so power users can discover them.
  • Closing footnote distinguishing internal-only verbs (kept callable but undocumented) from deprecated verbs (turbo, split, adapt — print banners pointing to replacements).

What's deliberately omitted

  • Exhaustive flag tables. Every section ends with Full flag list: reflex <verb> --help so this doc can't drift from source.
  • Internal-only diagnostic verbs: bench, bench-game, targets, guard, check, calibrate, status, config show/set, validate-legacy, validate-dataset.

Closes / supersedes

Test plan

  • Every verb listed is a real @app.command() in src/reflex/cli.py with hidden=False (verified via grep -n "@app\.command" src/reflex/cli.py cross-referenced against the live reflex --help output)
  • Every "advanced" verb (ros2-serve, replay) is @app.command(hidden=True) but otherwise production code paths
  • --state-msg-type flag matches fix(ros2): pluggable state extractors (joint_state / imu / odom) #133's implementation
  • Vertical matrix verticals all appear in 02_research/customers/MASTER.md as P0 tier
  • Live preview by mkdocs serve or equivalent (no docs site build wired up in this repo — the file is GitHub-rendered Markdown)

Adds docs/cli_reference.md — a single user-facing reference for every
visible reflex command. Audited against src/reflex/cli.py so every verb,
flag, and example matches the live source. Examples for each top-level
verb are real (no fabricated flag combinations).

Drops fabricated content from the original attempt (#127) — the
deprecated 'reflex turbo' verb, the internal-only 'reflex bench',
the never-existed 'AGV / GPS-fused state' guidance — and replaces it
with content grounded in the FastCrest customer research vault: the
Vertical Quick-Start Matrix now mirrors the P0 customer subcategories
(warehouse AMR, autonomous tractors, mining, drone surveillance,
last-mile drone delivery, traffic management, smart-camera retail/
warehouse, ADAS/autonomous trucking, port inspection, baggage tugs).

Documents the new --state-msg-type flag from #133 alongside the
recommended drone state topics (/mavros/local_position/odom for full
state, /mavros/imu/data for orientation-only fallback).

Supersedes #127.

Co-Authored-By: Divyansh Rawat <186957976+DsThakurRawat@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rylinjames rylinjames merged commit e2d2be4 into main May 15, 2026
4 checks passed
@rylinjames rylinjames deleted the docs/cli-command-reference branch May 15, 2026 21:41
rylinjames added a commit to DsThakurRawat/reflex-vla that referenced this pull request May 19, 2026
The existing 10 m/s ceiling capped end-effector velocity, which made
sense for arms but blocked legitimate aerial flight speeds. Quadcopter
and delivery drones routinely operate at 15-50 m/s (54-180 km/h).

Raises max from 10.0 -> 50.0 and polishes the description to make the
dual-use semantics explicit. None of the four shipped presets are
affected (franka 1.0, ur5 1.0, so100 0.5, quadcopter 5.0 — all well
under both ceilings).

Originally proposed by @DsThakurRawat in FastCrest#119; this commit applies just
the actual schema delta on top of current main, since the PR branch
predated FastCrest#132 (quadcopter preset) and FastCrest#134-FastCrest#138 — naively merging would
have reverted that work.

Co-Authored-By: DsThakurRawat <divyanshrawatofficial@gmail.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
rylinjames added a commit that referenced this pull request May 19, 2026
…nts (#119)

The existing 10 m/s ceiling capped end-effector velocity, which made
sense for arms but blocked legitimate aerial flight speeds. Quadcopter
and delivery drones routinely operate at 15-50 m/s (54-180 km/h).

Raises max from 10.0 -> 50.0 and polishes the description to make the
dual-use semantics explicit. None of the four shipped presets are
affected (franka 1.0, ur5 1.0, so100 0.5, quadcopter 5.0 — all well
under both ceilings).

Originally proposed by @DsThakurRawat in #119; this commit applies just
the actual schema delta on top of current main, since the PR branch
predated #132 (quadcopter preset) and #134-#138 — naively merging would
have reverted that work.

Co-authored-by: RomirJ <playindus@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

[Docs] Add Comprehensive CLI Command Reference

1 participant