Skip to content

Clarify README development and eval requirements - #9

Merged
ronniegeraghty merged 2 commits into
mainfrom
ronniegeraghty-readme-requirements
Nov 10, 2026
Merged

ronniegeraghty merged 2 commits into
mainfrom
ronniegeraghty-readme-requirements

Conversation

@ronniegeraghty

@ronniegeraghty ronniegeraghty commented Nov 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Closes #8.

Rewrite only the README Requirements section as the approved shared checklist for marketplace development and evals:

Use the requested introduction, “For marketplace development and evals, use this shared setup.” Preserve the local-install validator link and remove the redundant installed-plugin disclaimer, upstream runner note, and opt-in reminder from Requirements. The existing Use the marketplace section already covers installation and remains unchanged. Minima follow the starter-pinned upstream runner requirements.

Verified provisioning: .github/copilot/settings.json enables only marketplace-maintainer and plugin-authoring after checkout trust in supported hosts. marketplace.json lists the remote Evals plugin but does not enable it. Evals still requires explicit installation/enablement; npm development dependencies supply Copilot CLI and Vally, not the Evals plugin. No lifecycle script or tracked hook provisions Evals.

No dependency, runtime, manifest, or settings changes; all README content outside Requirements is unchanged.

Validation

  • git diff --check — passed.
  • Focused documentation assertions on the revised candidate — passed: exact requested introduction and paragraph removal, all five prerequisites unchanged, and no edits outside Requirements.
  • Provisioning assertions — passed: only the two maintenance plugins are enabled in checkout settings; Evals is catalog-only, with no npm lifecycle or tracked hook installer.
  • Initial npm run lint passed on the first revision in a clean tracked-files snapshot using existing dependencies: 6 skills and 7 eval specs passed. In-place lint was blocked by an obsolete empty plugin directory held open by another process; that process was left untouched and the snapshot was removed. Lint was not rerun for this prose-only follow-up.
  • Paid evals were not run.

Closes #8

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ronniegeraghty
ronniegeraghty requested a review from a team as a code owner November 10, 2026 20:10
Copilot AI auto review requested due to automatic review settings November 10, 2026 20:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The reviewed README update preserves installation and host guidance while documenting the relevant development and evaluation prerequisites, with no active review issues identified.

No open issues 🎉

What changed in this PR

Updates the README Requirements section with a shared checklist for marketplace development and CLI-based evals while preserving installation guidance, host-specific references, and opt-in Evals behavior.

Changes:

  • Documents supported Copilot harnesses, GitHub Copilot plan, Node.js/npm, Copilot CLI, Vally, and Git minimums.
  • Clarifies that installed plugins do not require the full development setup.
  • Retains local-install and upstream runner links.
File Description
README.md Rewrites the Requirements section with the approved development and eval prerequisites and clarifies installation scope.

Review effort: Lite (auto)

Remove the redundant runner note while preserving the approved prerequisites and existing opt-in Evals installation guidance.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ronniegeraghty
ronniegeraghty merged commit 34b8749 into main Nov 10, 2026
5 checks passed
@ronniegeraghty
ronniegeraghty deleted the ronniegeraghty-readme-requirements branch November 10, 2026 20:22
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.

Clarify the starter README requirements

2 participants