Skip to content

chore(external): bump modern-web-guidance plugin to 1.0.0 - #6365

Merged
aaronpowell merged 1 commit into
github:mainfrom
schiller-manuel:bump-modern-web-guidance-v1
Nov 13, 2026
Merged

aaronpowell merged 1 commit into
github:mainfrom
schiller-manuel:bump-modern-web-guidance-v1

Conversation

@schiller-manuel

Copy link
Copy Markdown
Contributor

Description

Bump the existing modern-web-guidance external plugin from 0.0.179 to 1.0.0, following the update process for previously approved listings. Original listing: #2533.

Source ref moves to v1.0.0; the upstream plugin.json at that tag declares version 1.0.0.

plugins/external.json is hand-edited; .github/plugin/marketplace.json is regenerated with npm run plugin:generate-marketplace.

Type of Contribution

  • Update to an existing plugin.

Pull Request Checklist

  • I have read and followed the contribution guidelines.
  • The changes follow the existing format and naming conventions.
  • This PR targets main.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@schiller-manuel
schiller-manuel requested a review from a team as a code owner November 11, 2026 07:14
Copilot AI dynamic review requested due to automatic review settings November 11, 2026 07:14
@github-actions github-actions Bot added external-plugin Public external plugin submission plugin PR touches plugins labels Nov 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 2 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

Copy link
Copy Markdown
Contributor

🟡 Contributor Reputation Check: MEDIUM risk

Check Risk
Profile MEDIUM
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions github-actions Bot added needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk merge-risk:high labels Nov 11, 2026
@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:high — Privileged execution, automation, or review-policy change
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access, including a maintainer with admin or maintain permission.

Why this tier
  • plugins/external.json is a high-risk path (automation, scripts, MCP config, hooks, or review policy)

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Plugin and extension validation ✅ Passed Passed · logs
Plugin structure ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs
Canvas/plugin smoke test ✅ Passed Passed · logs

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s); an approval from a maintainer with admin or maintain permission
  • The core-maintainers pool is not staffed yet; an approver with admin or maintain permission is required instead.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for 835b269 · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions github-actions Bot added the requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review label Nov 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🛑 External plugin PR checks failed (submitter fixes required)

  • Changed entries detected: 1
  • Workflow state label: requires-submitter-fixes
  • Full logs: Download quality gate artifact
  • Status legend: ✅ pass · ⚠️ warning · 🛑 fail

Per-plugin quality summary

Plugin spec compliance vally lint install smoke test version match ref/sha consistency canvas structure overall source tree
modern-web-guidance ✅ pass 🛑 fail ✅ pass ✅ pass ⚪ not_run ⚪ not_run 🛑 fail v1.0.0

Gate output details

modern-web-guidance - spec compliance (✅ pass)

Agent Plugins v1.0.0 manifest checks passed for plugin.json.
modern-web-guidance - vally lint (🛑 fail)

❌ chrome-extensions (1/2 checks passed, 1 failed)
    ✗ [spec-compliance] 1 of 1 skill(s) have spec violations.
        ✗ spec-compliance: Spec checks failed.
            ✗ File length (531 lines) exceeds limit (500).
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.
✅ modern-web-guidance (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

2 skill(s) linted, 1 passed, 1 failed
modern-web-guidance - install smoke test (✅ pass)

Install smoke test succeeded. Verified /tmp/external-plugin-quality-vLRuDQ/copilot-home/.copilot/installed-plugins/external-plugin-intake/modern-web-guidance/plugin.json.
modern-web-guidance - version match (pass)

- v1.0.0: matched version "1.0.0" at "plugin.json".
modern-web-guidance - ref/sha consistency (not_run)

Ref/SHA consistency gate skipped because one of source.ref or source.sha was not provided.
modern-web-guidance - canvas structure (not_run)

Canvas structure gate skipped because plugin is not tagged with "canvas".
- modern-web-guidance: spec=pass, vally-lint=fail, install-smoke=pass, version-match=pass, ref-sha-consistency=not_run, canvas-structure=not_run, overall=fail

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The major-version external content change lacks provided spec, lint, install-smoke, and version-match quality-gate results, so those checks must be confirmed before approval.

No open issues 🎉

What changed in this PR

Updates the approved modern-web-guidance external plugin listing to upstream release v1.0.0, synchronizing the repository metadata with the new source tag.

Changes:

  • Bumps the plugin version from 0.0.179 to 1.0.0.
  • Updates the source reference from v0.0.179 to v1.0.0.
  • Regenerates the marketplace metadata with matching values.
File Description
plugins/​external.json Updates the external plugin version and upstream release reference.
.github/​plugin/​marketplace.json Synchronizes the generated marketplace metadata with the updated plugin listing.

Review effort: Lite (auto)

@aaronpowell aaronpowell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the update. As you'll note, the checks are failing with a skill exceeding the 500 line limit of the SKILL.md spec. The length is only a minor amount over, so I'll merge as is, but it would be worth evaluating the skill length to whether there's an opportunity to simplify it or maybe split across some reference files.

@aaronpowell
aaronpowell merged commit 3c75cad into github:main Nov 13, 2026
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-plugin Public external plugin submission merge-risk:high needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk plugin PR touches plugins requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants