Skip to content

chore: bump GAF for TOON scanner summaries - #7268

Closed
robertolopezlopez wants to merge 1 commit into
chore/CLI-1859-gaf-toonfrom
feat/CLI-1860
Closed

robertolopezlopez wants to merge 1 commit into
chore/CLI-1859-gaf-toonfrom
feat/CLI-1860

Conversation

@robertolopezlopez

@robertolopezlopez robertolopezlopez commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages are release-note ready
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

  • Pins github.com/snyk/go-application-framework to the latest feat/CLI-1860 pseudo-version in cliv2 and cliv2-private.

Depends on snyk/go-application-framework#767 and #7267

Where should the reviewer start?

cliv2/go.mod and cliv2-private/go.mod — dependency-only change.

How should this be manually tested?

After building the CLI against this branch, run a scan with --toon (and optionally --toon-file) and confirm output includes scanner summaries and context fields per the updated GAF TOON schema.

What's the product update that needs to be communicated to CLI users?

None in this PR (dependency bump only). TOON output shape changes ship via GAF; user-facing notes belong with the GAF release and any follow-up CLI wiring.

Risk assessment

Low — go.mod / go.sum only. TOON-related pkg/core tests pass locally.

What are the relevant tickets?

CLI-1860

Test plan

  • go test ./cliv2/pkg/core/... -run TOON
  • CI green on this PR

Made with Cursor

@robertolopezlopez
robertolopezlopez requested a review from a team as a code owner September 15, 2026 21:17
@snyk-io

snyk-io Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@robertolopezlopez
robertolopezlopez marked this pull request as draft September 15, 2026 21:18
@snyk-pr-review-bot

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 5 relevant code sections from 4 files (average relevance: 0.98)

🤖 Repository instructions applied (from AGENTS.md)

@robertolopezlopez
robertolopezlopez changed the base branch from main to chore/CLI-1859-gaf-toon September 16, 2026 08:52
@robertolopezlopez
robertolopezlopez added this pull request to stack #7269 September 16, 2026 08:52
@robertolopezlopez robertolopezlopez changed the title chore: bump GAF for TOON scanner summaries [CLI-1860] chore: bump GAF for TOON scanner summaries Sep 16, 2026
@robertolopezlopez
robertolopezlopez removed this pull request from stack #7269 September 16, 2026 10:24
@robertolopezlopez
robertolopezlopez added this pull request to stack #7272 September 16, 2026 10:27
@robertolopezlopez
robertolopezlopez force-pushed the chore/CLI-1859-gaf-toon branch 2 times, most recently from 8a3a13d to 8d92239 Compare September 16, 2026 13:20
@robertolopezlopez
robertolopezlopez force-pushed the feat/CLI-1860 branch 2 times, most recently from c752c58 to 33b95af Compare September 16, 2026 13:36
@robertolopezlopez
robertolopezlopez force-pushed the chore/CLI-1859-gaf-toon branch 2 times, most recently from fbb4f1c to 40a6989 Compare September 16, 2026 14:16
@github-actions

github-actions Bot commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor
Fails
🚫

"func: TOON scanner summaries" is not using a valid commit message format. For commit guidelines, see: CONTRIBUTING.

Generated by 🚫 dangerJS against 459e2a2

@robertolopezlopez
robertolopezlopez force-pushed the feat/CLI-1860 branch 2 times, most recently from cb387ab to 41f9135 Compare September 16, 2026 18:15
@octavian-snyk

octavian-snyk commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Good to go once GAF #767 is merged and the version is updated. There's no other changes this PR introduced.

@robertolopezlopez

Copy link
Copy Markdown
Contributor Author

Superseded: GAF pin bump folded into #7274, which now targets chore/CLI-1859-gaf-toon directly.

@robertolopezlopez
robertolopezlopez removed this pull request from stack #7272 September 17, 2026 10:21
@robertolopezlopez
robertolopezlopez deleted the feat/CLI-1860 branch September 17, 2026 10:23
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