Skip to content

Link to CLI docs from README#147

Merged
jonathanpopham merged 2 commits intomainfrom
add-cli-docs-link
Apr 29, 2026
Merged

Link to CLI docs from README#147
jonathanpopham merged 2 commits intomainfrom
add-cli-docs-link

Conversation

@jonathanpopham
Copy link
Copy Markdown
Contributor

@jonathanpopham jonathanpopham commented Apr 29, 2026

Summary

  • Adds a prominent link to the new CLI docs site (docs.supermodeltools.com/cli/overview) near the top of the README
  • Adds a CLI Docs row to the Links table

Companion PR on the docs repo: supermodeltools/docs#7

Test plan

Summary by CodeRabbit

  • Documentation
    • README updated to include direct links to the full CLI documentation in multiple visible locations for easier access.
    • Added a standalone "Full CLI docs" callout near the top and an additional "CLI Docs" entry in the Links table to improve discoverability.

Adds the new docs.supermodeltools.com/cli home as a prominent link near
the top of the README and as a row in the Links table. The docs cover
install, file mode, agent integration, configuration, and a full
command reference — see supermodeltools/docs#7.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 50f90391-2fed-4c9c-886e-648ec9c555eb

📥 Commits

Reviewing files that changed from the base of the PR and between 63240e8 and 473202d.

📒 Files selected for processing (1)
  • README.md
✅ Files skipped from review due to trivial changes (1)
  • README.md

Walkthrough

The README.md was updated to add direct links to the full CLI documentation in two places: a new standalone “Full CLI docs” line near the top and an additional “CLI Docs” entry in the Links table. No functional code or exported entities were changed.

Changes

Cohort / File(s) Summary
Documentation
README.md
Inserted two direct references to the full CLI documentation: a standalone “Full CLI docs” link near the top and an additional “CLI Docs” row in the Links table. No other content modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • greynewell

Poem

README gains a little door,
CLI guides now shown once more,
Two small links that point the way,
Find the docs without delay. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding links to CLI documentation in the README.
Description check ✅ Passed The description covers the core changes and includes a test plan, but diverges from the template by using a 'Summary' section instead of 'What', 'Why', and 'Test plan' sections.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-cli-docs-link

Review rate limit: 2/5 reviews remaining, refill in 29 minutes and 59 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
README.md (1)

245-245: Minor UX nit: keep the link label wording consistent.

Top callout uses “Full CLI docs” (Line 7), while the Links table row uses “CLI Docs” (Line 245). Consider using the same label in both places (either “CLI Docs” everywhere or “Full CLI docs” everywhere) to reduce cognitive friction.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 245, The README has inconsistent link label wording: the
top callout uses "Full CLI docs" while the Links table uses "CLI Docs" (the
table entry linking to https://docs.supermodeltools.com/cli/overview). Update
the table row label (or the top callout) so both labels match exactly—e.g.,
change the table's "CLI Docs" to "Full CLI docs" (or change the top callout to
"CLI Docs")—and ensure the link target remains
https://docs.supermodeltools.com/cli/overview.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@README.md`:
- Line 245: The README has inconsistent link label wording: the top callout uses
"Full CLI docs" while the Links table uses "CLI Docs" (the table entry linking
to https://docs.supermodeltools.com/cli/overview). Update the table row label
(or the top callout) so both labels match exactly—e.g., change the table's "CLI
Docs" to "Full CLI docs" (or change the top callout to "CLI Docs")—and ensure
the link target remains https://docs.supermodeltools.com/cli/overview.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2a9e240a-458b-400c-bba1-35c9b7a2205a

📥 Commits

Reviewing files that changed from the base of the PR and between 0b86a59 and 63240e8.

📒 Files selected for processing (1)
  • README.md

@jonathanpopham jonathanpopham merged commit 66c1123 into main Apr 29, 2026
7 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