Skip to content

chore(fullsend): re-pin triage-security to its main commit c4891346 - #345

Merged
mrizzi merged 1 commit into
mainfrom
TC-6931-repin
Oct 15, 2026
Merged

mrizzi merged 1 commit into
mainfrom
TC-6931-repin

Conversation

@mrizzi

@mrizzi mrizzi commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • PR feat(triage-security): add discover entry point for untriaged Vulnerabilities #344 was rebase-merged, so the triage-security pin 9fabe0ef is not an ancestor of main. It only resolves through refs/pull/344/head, which goes against the merge-commit rule in fullsend.md.
  • This re-pins the triage-security base: to c4891346, the same commit as it landed on main. The base hash is unchanged (110fb33d…). Only triage-security is re-locked.
  • No behaviour change: every pinned file hash in lock.yaml is the same. Only URLs, timestamps and the child-harness hash differ (the child file holds the new URL). .fullsend/harness/verify-pr.yaml and its lock entry are untouched.

Merge with a merge commit (not squash/rebase): the re-pinned c4891346 is already on main, but a rebase would rewrite this PR's own commit as well.

Test plan

  • git show c4891346:harness/triage-security.yaml | shasum -a 256 gives 110fb33d…, matching the pin
  • fullsend lock triage-security --fullsend-dir .fullsend succeeds; no 9fabe0ef left in lock.yaml
  • git diff main -- .fullsend/harness/verify-pr.yaml is empty

Refs TC-6931

🤖 Generated with Claude Code

Summary by Sourcery

Enhancements:

  • Re-pin the triage-security harness and its locked dependencies to the commit already present on main without changing pinned file content.

PR #344 was rebase-merged, so the pinned 9fabe0e is not on main and
only resolves through refs/pull/344/head. Re-pin the triage-security
base to c489134, the same content as it landed on main, and re-lock
triage-security only. Pinned file hashes are unchanged; verify-pr is
untouched.

Refs TC-6931

Assisted-by: Claude Code
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sourcery-ai

sourcery-ai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Reviewer's Guide

Re-pins triage-security to commit c489134, which is present on main, and regenerates only its lock metadata; content hashes and runtime behavior remain unchanged.

File-Level Changes

Change Details Files
Re-pin the triage-security harness and all resolved dependency URLs from the obsolete pull-request ref to the corresponding commit on main, while preserving content integrity.
  • Change the triage-security base URL from 9fabe0e to c489134 without changing its SHA-256 pin.
  • Regenerate triage-security lock metadata so dependency URLs use c489134 and timestamps reflect the relock.
  • Verify that all pinned file and directory hashes remain unchanged and leave verify-pr unchanged.
.fullsend/harness/triage-security.yaml
.fullsend/lock.yaml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot 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.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@fullsend-ai-review

fullsend-ai-review Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

🤖 Finished Verify Pr · ⏭️ Skipped (no Jira issue links PR #345 in its Git Pull Request field) · Started 12:11 PM UTC · Completed 12:11 PM UTC

Commit: 6bffb30 · View workflow run →

Effort: high

@mrizzi
mrizzi merged commit db9c8b1 into main Oct 15, 2026
18 checks passed
@mrizzi
mrizzi deleted the TC-6931-repin branch October 15, 2026 12:17
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.

1 participant