Skip to content

refactor(skills): draft ReadMe workflow-focused skill skeletons - #11

Draft
xavierandueza wants to merge 4 commits into
mainfrom
feature/improve-skills
Draft

xavierandueza wants to merge 4 commits into
mainfrom
feature/improve-skills

Conversation

@xavierandueza

@xavierandueza xavierandueza commented Oct 2, 2026 •

Copy link
Copy Markdown
🚥 Resolves N/A

🧰 Changes

This is a skeleton that I made for the Skills that we'll want moving forward. Other PRs merge into here, and once we're ready to go we merge this.

Note - it's intentionally minimalist at the moment so that we can continue to improve on this as we merge PRs that and we test to make sure that the skills work in actual onboarding workflows that we do.

🧬 QA & Testing

N/A - staging PR that others merge into.

@xavierandueza xavierandueza self-assigned this Oct 2, 2026
@greptile-apps

greptile-apps Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

[Low risk] Reorganizes documentation and skill guides for the ReadMe plugin.

The PR is not ready to merge because the client skill copies fail the required sync check.

Findings

  1. P1 Local seeding stops without MCP ▶

Reviews (4) · Last reviewed commit: "Improve canonical ReadMe skills"

Comment thread skills/PROJECT-WORKFLOW.md Outdated
Comment thread skills/setup-project/SKILL.md Outdated
@xavierandueza

Copy link
Copy Markdown
Author

(agent response)
Both findings are addressed: standalone project/API routing and connection verification before seeding. Local skill sync, frontmatter/size, whitespace, and local-reference checks passed. On this head, validate is successful and Greptile Review is in progress; the previous 3/5 confidence result was for the old head, so a fresh review is needed.

Comment thread skills/setup-project/SKILL.md Outdated
**Done:** The user approves the setup scope and which account/project to create or reuse.
3. **Provision.** Read [Creating a project](https://docs.readme.com/main/docs/creating-a-project) and discover available onboarding operations. Create the approved account/project only through supported capabilities. If unavailable, hand the user the signup link (`https://dash.readme.com/signup`) and the documented project-creation steps; resume when they return the project link. Let the user enter credentials, consent, and billing details.
**Done:** A confirmed project link and owner exist, or a specific manual handoff is recorded.
4. **Connect and verify.** Use the existing connection first. Only when connection setup is needed, read the [client-specific connection reference](../mcp-auth/CLIENTS.md); apply non-secret configuration only with permission, and let the user supply credentials through their client's settings. Use a supported, non-mutating project read and inspect its current schema to confirm the connection reaches the intended project. Invoke [mcp-auth](../mcp-auth/SKILL.md) only when the read fails for authentication, then retry the read after recovery. If the read is unavailable or does not succeed, stop before seeding and give the user a client-specific manual next step.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Local seeding stops without MCP

If a user has confirmed bidirectional Git sync but no working MCP connection, setup-project stops at step 4 before it reaches the local-file path in step 5. The user cannot seed the mapped files even though that work does not need a hosted write connection. Require the connection check for hosted writes, not for confirmed local edits.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

(agent response)
Moved write-surface selection before the hosted-access gate. Explicitly confirmed bidirectional Git-sync edits to mapped local files no longer require MCP or successful authentication; hosted writes still require a successful read of the intended project. Mixed setup keeps blocked hosted-only items pending without falling back to local files. Updated the downstream helpers, client copies, and catalogs; local checks and independent static verification pass (no live integrations claimed).

Commit: c0fc720

@xavierandueza
xavierandueza force-pushed the feature/improve-skills branch from c0fc720 to c1fae3d Compare October 2, 2026 07:30
@xavierandueza

Copy link
Copy Markdown
Author

(agent response)
Validate job 110748856324 fails at node scripts/sync-skills.mjs --check: 51 mirror mismatches, exit 1, reproduced locally on c1fae3d.

This is intentional staging: this PR changes only skills/; mirrored copies and CI remain untouched per the requested scope. Independent canonical-only validation passes for all six skills; later CI steps were skipped, not passed. No product changes or CI rerun: CI remains red until mirrors are synced at go-live or owners separately change the sync policy.

@xavierandueza

Copy link
Copy Markdown
Author

(agent response)
Greptile completed successfully, but its check remains red: 4/5 is below the required 5/5. The latest summary cites client-copy synchronization and carries the old, already-replied/outdated local-seeding finding. On c1fae3d, canonical setup steps 4–6 route first and explicitly skip MCP for confirmed local-only work; no new canonical finding was posted.

No product changes: this draft intentionally stages only skills/; mirrors stay unchanged until go-live. Independent canonical validation passes all six skills. Neither CI nor the confidence gate is green; no rerun, threshold change, or thread-resolution action was taken.

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