Skip to content

chore(fullsend): update to v0.55.0 - #3686

Open
jsmid1 wants to merge 1 commit into
conforma:mainfrom
jsmid1:EC-2390
Open

jsmid1 wants to merge 1 commit into
conforma:mainfrom
jsmid1:EC-2390

Conversation

@jsmid1

@jsmid1 jsmid1 commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Update Fullsend and the pinned agent harnesses to v0.55.0. Add custom code and fix harnesses that enable GitHub-signed commits, and migrate the issue-target allowlist to the current configuration schema.

Resolves: EC-2390

Update Fullsend workflow refs and pinned agent harnesses to v0.55.0. Enable GitHub-signed commits for the code and fix agents, and migrate the issue-target allowlist to the current configuration schema.

Co-authored-by: Codex <noreply@openai.com>
@jsmid1
jsmid1 requested a review from a team as a code owner November 11, 2026 11:52
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review.

Walkthrough

The change adds code and fix harnesses, updates harness base references and repository target settings, and changes the reusable workflow commit used by the dispatch and prioritize jobs.

Changes: Harness configuration

Layer / File(s) Summary
Add harnesses and update base references
.fullsend/customized/harness/code.yaml, .fullsend/customized/harness/fix.yaml, .fullsend/customized/harness/review.yaml, .fullsend/customized/harness/retro.yaml
Adds code and fix harness configurations with GitHub signed-commit overlays. Updates the base references for review and retro.
Register harnesses and update repository targets
.fullsend/config.yaml
Adds code and fix to the agents list. Moves the repository allowlist to tracker.github.allowed_targets.repos without changing its entries.

Reusable workflow pins

Layer / File(s) Summary
Update reusable workflow references
.github/workflows/fullsend.yaml, .github/workflows/prioritize.yaml
Updates both jobs to reference reusable workflows at commit 87e94fba1dd0203a373f6e992269f9ccfe81e471.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: st3penta, robnester-rh

Merge Risk: Minimal · up to f3327

This PR updates Fullsend pins, enables GitHub-signed commits for the code and fix agents, and migrates the target allowlist to the current schema. The caller and callee contracts checked in earlier inspections are compatible, and no concrete merge-blocking risk remains.

Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed The title clearly identifies the Fullsend version update, which is the main change in the pull request.
Description check Passed The description explains the update, signed-commit harness changes, configuration migration, and linked ticket. It omits the template headings but includes the required information.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (7 skipped: 7 unsupported.)

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b59f9594-f11f-462e-8133-c1bd7280c9c5
Commits

Reviewing files that changed from the base of the PR and between 3bc55d6 and f332719.

Files selected for processing: 7

  • .fullsend/config.yaml
  • .fullsend/customized/harness/code.yaml
  • .fullsend/customized/harness/fix.yaml
  • .fullsend/customized/harness/retro.yaml
  • .fullsend/customized/harness/review.yaml
  • .github/workflows/fullsend.yaml
  • .github/workflows/prioritize.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
acceptance 54.61% <ø> (ø)
generative 12.22% <ø> (ø)
integration 23.62% <ø> (ø)
unit 70.87% <ø> (-1.43%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 121 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fullsend-ai-review fullsend-ai-review Bot added the risk/moderate PR risk: moderate label Nov 11, 2026
@fullsend-ai-review

Copy link
Copy Markdown

<!-- fullsend:risk-assessment -->
Risk Assessment: moderate (2/5)

Details

A routine Fullsend version bump to v0.55.0 touching only configuration YAML files; protected-path count (2) and CI workflow modification each push individual signals high, but very low recent churn (1 commit/30d) and no security-sensitive changes keep the weighted composite moderate.

@fullsend-ai-review

Copy link
Copy Markdown

Review

Findings

Medium

  • [protected-path] .github/workflows/fullsend.yaml, .github/workflows/prioritize.yml — PR modifies files under protected paths (.github/). These are governance/infrastructure files that require human approval. The PR provides sufficient justification (linked to EC-2390, description explains version upgrade to v0.55.0), but human approval is always required for protected-path changes regardless of context.

Labels: PR modifies GitHub Actions workflow files and CI/agent configuration

<!-- fullsend:review-findings-v2:eyJ2ZXJzaW9uIjoyLCJmaW5kaW5ncyI6W119 -->

@fullsend-ai-review fullsend-ai-review Bot added requires-manual-review Review requires human judgment github_actions Pull requests that update GitHub Actions code ci labels Nov 11, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci github_actions Pull requests that update GitHub Actions code requires-manual-review Review requires human judgment risk/moderate PR risk: moderate size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants