Skip to content

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

Merged
xavierandueza merged 61 commits into
mainfrom
feature/improve-skills
Oct 13, 2026
Merged

xavierandueza merged 61 commits into
mainfrom
feature/improve-skills

Conversation

@xavierandueza

@xavierandueza xavierandueza commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor
🚥 Resolves N/A

NOTE - ONLY LOOK AT the managing-readme-docs REST WILL BE UPDATED IN OTHER PRs.

🧰 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 impact] Reorganizes skill documentation and configuration files.

Confirm that a repository is connected through BiDi Sync before merging; otherwise requested hosted edits can remain local.

Findings

  1. P1 Hosted docs stay unchanged ▶

Reviews (5) · Last reviewed commit: "chore: sync generated skills" · Reviewed by Greptile

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

Copy link
Copy Markdown
Contributor 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
@xavierandueza
xavierandueza force-pushed the feature/improve-skills branch from c0fc720 to c1fae3d Compare October 2, 2026 07:30
@xavierandueza

Copy link
Copy Markdown
Contributor 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
Contributor 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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the ONLY one that has work in it this PR - IGNORE THE REST.

xavierandueza and others added 25 commits October 12, 2026 15:19
…kill

feat(project-creation): RM-18984 Create Project Skill
…settings-skill

feat(update-project-settings): RM-18985 Add project settings skill for PATCH /v2/projects/me
@xavierandueza
xavierandueza marked this pull request as ready for review October 13, 2026 06:11

## Documentation Management Mechanism

If you are in a documentation repository, use git to manage documentation via BiDi Sync unless the user has specified otherwise. Use MCP if you are not in a docs repository.

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 Hosted docs stay unchanged

If the agent is in a repository containing docs/ or reference/ that has never been connected to ReadMe, this instruction still chooses Git for a request to update the hosted docs. The edits stay local, and the requested site remains unchanged.

Confirm that BiDi Sync connects this repository and its branch to the intended project before choosing local edits; otherwise use MCP. The same instruction appears at line 46 and in the Claude, Codex, and Cursor copies.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@xavierandueza
xavierandueza merged commit 35177f4 into main Oct 13, 2026
1 of 2 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.

2 participants