Skip to content

docs: list Kiro Pi and Muse in provider overview - #31046

Open
jacobcorbett wants to merge 1 commit into
pingdotgg:mainfrom
jacobcorbett:docs/list-supported-providers
Open

jacobcorbett wants to merge 1 commit into
pingdotgg:mainfrom
jacobcorbett:docs/list-supported-providers

Conversation

@jacobcorbett

Copy link
Copy Markdown
Contributor

Problem

The README provider overview and installation list omit Kiro, Pi, and Muse Code even though they have supported drivers and setup guides. Readers can mistake those providers for unsupported integrations.

Change

Add the three providers to both overview lists and link their existing setup guides beside their login commands.

Scope and approval

This is a focused correction of existing documentation to match the implementation, under the contribution guide's small obvious-fix exception. It changes no product behavior and addresses one underlying documentation problem.

Verification

  • Compared the commands with docs/user/providers-kiro.md, providers-pi.md, providers-muse.md and the provider table in docs/user/install.md. Confirmed the three packages export their server drivers and all new local guide links resolve.
  • Local findings-first review found no blocking issues; the branch contains only the described documentation correction.
  • vp fmt --check README.md with repository-pinned Vite+ 1.4.0: passed.
  • git diff --check origin/main HEAD and local Markdown link/anchor checks for changed documents: passed.
  • The broader vp run check:changed runner could not start in this fresh checkout because workspace dependencies, beginning with @effect/platform-node, are absent. No application tests or live UI checks were run for this documentation-only change.

Model: GPT-6.1 Sol. Harness: Codex, through T3 Code.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T17:59:34.416506Z a9b4e18 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Nov 4, 2026
@macroscopeapp

macroscopeapp Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at a9b4e18

Macroscope's review found this PR approvable — This README update lists Kiro, Pi, and Muse Code and links to their existing setup guides. It changes documentation only, without altering product defaults or runtime behavior.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The README now lists Kiro, Pi, and Muse Code as supported providers and adds their installation and authentication instructions with setup guide links. The existing fx installation instructions remain.

Changes: Provider documentation

Layer / File(s) Summary
Supported providers and setup instructions
README.md
The README adds Kiro, Pi, and Muse Code to its supported-subscription list and installation warning. The warning includes installation and authentication instructions and links to setup guides. The existing fx instructions remain.

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Possibly related PRs

Suggested labels: size:XS, vouch:unvouched

Suggested reviewers: darlingtondeveloper, macroscopeapp

Merge Risk: Minimal · up to a9b4e

This documentation-only change matches the repository’s provider setup guidance and has no identified merge-blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title "docs: list Kiro Pi and Muse in provider overview" matches the changeset. The raw summary says the README provider lists now include Kiro, Pi, and Muse Code, and the installation warning add…
Description check Passed The description follows the repository template's main sections. It explains the problem (README omits Kiro, Pi, and Muse Code despite supported drivers and guides) and the change (adds the three prov…
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.
Full details: Title check

Explanation

The title "docs: list Kiro Pi and Muse in provider overview" matches the changeset. The raw summary says the README provider lists now include Kiro, Pi, and Muse Code, and the installation warning adds setup guide links for those providers. The title is concise, uses a conventional commit prefix, and names the primary change. It is slightly informal in naming "Muse" rather than "Muse Code", but the meaning is clear.

Full details: Description check

Explanation

The description follows the repository template's main sections. It explains the problem (README omits Kiro, Pi, and Muse Code despite supported drivers and guides) and the change (adds the three providers to both overview lists and links setup guides beside login commands). The Scope and approval section explains why the change qualifies as a small documentation fix without a prior issue. The Verification section lists specific checks and results, and it states what was not run (application tests and live UI checks). The description does not include a commit-style title line, which the template comment suggests, but the title is provided separately. The model and harness are named at the end, as the template asks.

Finishing Touches
Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between dd5f98e and a9b4e18.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant