Skip to content

[external-plugin] Update sumo-logic plugin keywords - #3217

Open
vsinghal13 wants to merge 3 commits into
github:mainfrom
vsinghal13:update-sumo-logic-sha
Open

vsinghal13 wants to merge 3 commits into
github:mainfrom
vsinghal13:update-sumo-logic-sha

Conversation

@vsinghal13

@vsinghal13 vsinghal13 commented Sep 16, 2026

Copy link
Copy Markdown

Summary

  • Bump sumo-logic plugin version to 1.0.1 with new tag v1.0.1
  • Add keywords: devops, search, security
  • Regenerate marketplace.json

Details

Copilot AI balanced review requested due to automatic review settings September 16, 2026 18:51
@github-actions github-actions Bot added external-plugin Public external plugin submission plugin PR touches plugins labels Sep 16, 2026
@github-actions

github-actions Bot commented Sep 16, 2026

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 github-actions Bot added the ready-for-review Submission passed intake validation and is ready for maintainer review label Sep 16, 2026
@github-actions

github-actions Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

✅ External plugin PR checks passed

  • Changed entries detected: 1
  • Workflow state label: ready-for-review
  • Status legend: ✅ pass · ⚠️ warning · 🛑 fail

Per-plugin quality summary

Plugin spec compliance (non-blocking) vally lint install smoke test version match ref/sha consistency canvas structure overall source tree
sumo-logic ✅ pass ✅ pass ✅ pass ✅ pass ✅ pass ⚪ not_run ✅ pass f905b914984fabb5e0571290eafd3013c277d9b1

Gate output details

sumo-logic - spec compliance (✅ pass)

Agent Plugins v1.0.0 manifest checks passed for plugin.json.
sumo-logic - vally lint (✅ pass)

✅ sumo-investigator (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.
✅ sumosetup (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, 2 passed
sumo-logic - install smoke test (✅ pass)

Install smoke test succeeded. Verified /tmp/external-plugin-quality-JSZQ0l/copilot-home/.copilot/installed-plugins/external-plugin-intake/sumo-logic/plugin.json.
sumo-logic - version match (pass)

- v1.0.1: matched version "1.0.1" at "plugin.json".
- f905b914984fabb5e0571290eafd3013c277d9b1: matched version "1.0.1" at "plugin.json".
sumo-logic - ref/sha consistency (pass)

source.ref "v1.0.1" resolves to the same commit as source.sha "f905b914984fabb5e0571290eafd3013c277d9b1".
sumo-logic - canvas structure (not_run)

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

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.

🟡 Changes recommended

The new keywords and regenerated marketplace output are missing.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Updates the pinned Sumo Logic plugin revision to target its latest metadata.

Changes:

  • Replaces the pinned commit SHA for sumo-logic.
File summaries
File Description
plugins/external.json Updates the Sumo Logic plugin source SHA.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread plugins/external.json Outdated

@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.

A SHA change requires the version bumped for a plugin, I would also expect that the ref has changed as the ref should be treated as immutable (yes, you can force push to a tag so it's not truly immutable).

Also, the marketplace.json needs to be regenerated with the changes.

Copilot AI review requested due to automatic review settings September 17, 2026 02:40

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.

🟡 Changes recommended

The committed source ref and SHA conflict with the artifact identified in the PR description.

Get a fresh assessment by requesting another Copilot review.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread plugins/external.json
…etplace

- Version: 1.0.0 → 1.0.1
- Ref: v1.0.0 → v1.0.1
- SHA: updated to match v1.0.1 tag
- Keywords: added devops, search, security (removed siem)
- Regenerated marketplace.json

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings September 17, 2026 02:43
@vsinghal13
vsinghal13 force-pushed the update-sumo-logic-sha branch from fc745d8 to f588b3a Compare September 17, 2026 02:43
@github-actions github-actions Bot added requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review and removed ready-for-review Submission passed intake validation and is ready for maintainer review labels Sep 17, 2026

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 source ref and SHA contradict the exact values stated in the PR description.

Review details

Suppressed comments (1)

plugins/external.json:1278

  • The immutable source locator contradicts the PR description, which says the ref remains v1.0.0 and the new SHA is 607c742cfb18ca8a9b150b3c93589dad1a842345; this entry instead selects v1.0.1 at f905b914984fabb5e0571290eafd3013c277d9b1. Since these fields determine the exact plugin code users install, align the entry with the reviewed source or update the PR description and verify the new tag/SHA provenance before merging.
      "ref": "v1.0.1",
      "sha": "f905b914984fabb5e0571290eafd3013c277d9b1"
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@github-actions github-actions Bot added ready-for-review Submission passed intake validation and is ready for maintainer review and removed requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review labels Sep 17, 2026
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 plugin PR touches plugins ready-for-review Submission passed intake validation and is ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants