Skip to content

Preserve patch size metadata for local agent artifacts - #8136

Merged
chubes4 merged 1 commit into
mainfrom
fix/8118-opencode-empty-patch-size
Jul 14, 2026
Merged

Preserve patch size metadata for local agent artifacts#8136
chubes4 merged 1 commit into
mainfrom
fix/8118-opencode-empty-patch-size

Conversation

@chubes4

@chubes4 chubes4 commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

  • Finalized Homeboy agent-task cook run recovery-homeboy-8118-empty-patch-size into review-ready branch fix/8118-opencode-empty-patch-size.

Proof provenance

  • Proof runner: Homeboy agent-task cook loop
  • Proof ID: agent-task-finalization:recovery-homeboy-8118-empty-patch-size
  • Homeboy run ID: recovery-homeboy-8118-empty-patch-size
  • Manual proof: none recorded by this Homeboy finalization

Publication intent

  • Schema: homeboy/agent-task-publication-intent/v1
  • Action: review_request
  • Target kind: code_review
  • Adapter: github_pull_request
  • Base: main
  • Head: fix/8118-opencode-empty-patch-size

Source refs

Source relationship

  • Related finding ID: none recorded
  • Source packet ID: none recorded
  • Change kind: runtime-fix
  • Supersedes: none recorded
  • Depends on: none recorded

Runtime guardrails

  • Why this is not broader than the packet evidence: Only canonical regular files under the Homeboy-owned artifact root are measured; foreign, inaccessible, directory, and escaping paths remain unknown.
  • Evidence discriminators preserved: canonical Homeboy artifact root ownership
  • Nearby contracts preserved: unknown foreign artifact paths remain review-only

Attempt summary

Homeboy-owned local artifact files are measured before aggregation; zero-byte patches become no-op while inaccessible artifacts remain conservatively unknown.

Gate results

  • artifact-normalization: passed (targeted)
  • agent-task-aggregate: passed (targeted)
  • agent-task-provider: passed (targeted)
  • format: passed (targeted)
  • diff-check: passed (targeted)

Verification capability

  • targeted_checks_run: cargo test --lib homeboy_local_artifact_normalization_measures_empty_nonempty_and_unavailable_files, cargo test --lib core::agent_task_aggregate::tests -- --test-threads=1, cargo test --lib core::agent_task_provider::tests -- --test-threads=1, cargo fmt --check, git diff --check
  • targeted_checks_unavailable: none recorded
  • ci_expected: none recorded
  • manual_reviewer_check: none recorded

CI-equivalent coverage

  • CI-equivalent required gate: CI-equivalent required gate was not recorded; targeted proof must not be treated as full CI coverage

Changed files

  • src/core/agent_task_aggregate.rs
  • src/core/agent_task_provider.rs
  • src/core/agent_task_provider/command_runner.rs
  • src/core/agent_task_provider/outcome_normalization.rs
  • src/core/agent_task_provider/tests/outcome_tests.rs

Artifact refs

  • none recorded

Final status

  • Status: review-ready
  • Base: main
  • Head: fix/8118-opencode-empty-patch-size
  • Merge/deploy: not performed

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode
  • Model: openai/gpt-5.6-sol
  • Used for: Reproduced the artifact metadata regression, implemented the generic normalization boundary and tests, and verified the candidate with Chris.

@homeboy-ci

homeboy-ci Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — homeboy

Lint

review lint — passed

ℹ️ Full options: homeboy self docs commands/lint
ℹ️ Save lint baseline: homeboy review lint homeboy --baseline
Deep dive: homeboy review lint homeboy --changed-since df637ff

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-homeboy-review-lint-homeboy-Linux contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-homeboy-review-lint-homeboy-Linux contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/homeboy/actions/runs/29348054650

Audit

review audit — passed

  • structural — 59 finding(s)
  • core_boundary_leak:core-agnostic-source — 22 finding(s)
  • intra-method-duplication — 17 finding(s)
  • dead_code — 7 finding(s)
  • test_quality — 6 finding(s)
  • compiler — 5 finding(s)
  • near-duplication — 5 finding(s)
  • Types — 3 finding(s)
  • docs — 2 finding(s)
  • duplication — 2 finding(s)
  • Total: 136 finding(s)

Deep dive: homeboy review audit homeboy --changed-since df637ff

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-homeboy-review-audit-homeboy-Linux contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-homeboy-review-audit-homeboy-Linux contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/homeboy/actions/runs/29348054650
Tooling versions
  • Homeboy CLI: homeboy 0.284.8+7d1e24340a3e-dirty+7d1e24340
  • Extension: rust from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: c995082a
  • Action: unknown@unknown

@chubes4
chubes4 merged commit ca411a8 into main Jul 14, 2026
3 checks passed
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