chore: promote dev to main — agent v0.28.0 (Mantis #1216, audit COMPLIANT)

[Snider]

Closes Mantis #1216.

Summary by CodeRabbit

Release Notes

• Bug Fixes

  • Improved error handling and validation across HTTP operations, configuration parsing, and action execution.
  • Enhanced failure detection for stream operations and workspace cleanup tasks.
  • Fixed missing result validation in repository synchronisation and PR workflows.

• New Features

  • Added structured result-based variants for YAML unmarshalling and stream operations.

• Tests

  • Introduced comprehensive unit and example tests for agent compatibility, Forge API, and provider routing.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e5450642-5a96-4815-8547-7ea45940799e

📥 Commits

Reviewing files that changed from the base of the PR and between 39a2d41 and d40e919.

⛔ Files ignored due to path filters (1)

• go.work is excluded by !**/*.work

📒 Files selected for processing (39)

• go/cmd/core-agent/commands.go
• go/cmd/core-agent/main.go
• go/pkg/agentcompat/agentcompat.go
• go/pkg/agentcompat/agentcompat_example_test.go
• go/pkg/agentcompat/agentcompat_test.go
• go/pkg/agentic/actions.go
• go/pkg/agentic/auto_pr.go
• go/pkg/agentic/branch_cleanup.go
• go/pkg/agentic/commands_forge.go
• go/pkg/agentic/commands_workspace.go
• go/pkg/agentic/dispatch.go
• go/pkg/agentic/epic.go
• go/pkg/agentic/fleet_connect.go
• go/pkg/agentic/flow.go
• go/pkg/agentic/forge_client.go
• go/pkg/agentic/forge_client_example_test.go
• go/pkg/agentic/forge_client_test.go
• go/pkg/agentic/handlers.go
• go/pkg/agentic/mirror.go
• go/pkg/agentic/pipeline_budget.go
• go/pkg/agentic/pr.go
• go/pkg/agentic/prep.go
• go/pkg/agentic/provider_manager.go
• go/pkg/agentic/qa_analysis.go
• go/pkg/agentic/qa_cluster.go
• go/pkg/agentic/queue.go
• go/pkg/agentic/review_queue.go
• go/pkg/agentic/scan.go
• go/pkg/agentic/training_journal.go
• go/pkg/brain/actions.go
• go/pkg/brain/brain.go
• go/pkg/brain/direct.go
• go/pkg/brain/provider_contract.go
• go/pkg/brain/provider_contract_example_test.go
• go/pkg/brain/provider_contract_test.go
• go/pkg/lib/lib.go
• go/pkg/monitor/monitor.go
• go/pkg/runner/queue.go
• go/pkg/runner/runner.go

---

📝 Walkthrough

Walkthrough

The pull request introduces a structured error-handling pattern migration using core.Result across multiple packages. It adds Result-based variants for YAML unmarshalling and stream operations in the agentcompat package, converts the Forge HTTP boundary from error-based to Result-based returns, and improves error propagation by validating previously-ignored operation outcomes. New tests and examples are added for validation, whilst comment-only changes demonstrate usage of the new patterns.

Changes

Cohort / File(s)	Summary
agentcompat package – Result-based operations go/pkg/agentcompat/agentcompat.go, go/pkg/agentcompat/agentcompat_test.go, go/pkg/agentcompat/agentcompat_example_test.go	Adds UnmarshalYAMLResult, SendResult, ReceiveResult, and CloseResult methods returning core.Result for ConcurrencyLimit and HTTPStream types, with existing error-based methods becoming adapters that convert Result outcomes to errors. Introduces comprehensive unit tests and example tests validating YAML unmarshalling, HTTP roundtrip handling, error cases (missing fields, nil receivers), and response buffer management.
Forge client migration – HTTP boundary go/pkg/agentic/forge_client.go, go/pkg/agentic/forge_client_test.go, go/pkg/agentic/forge_client_example_test.go	Migrates HTTP helpers (doJSON, getJSON, postJSON, patchJSON, deletePath) and all downstream API methods to return core.Result instead of error. Adds forgeAPIError formatting validation tests, introduces APIError as a public alias, and adds an unused forgeResultError adapter for contract compatibility.
Agentic handlers – error propagation go/pkg/agentic/actions.go, go/pkg/agentic/handlers.go, go/pkg/agentic/dispatch.go	Updates QA/Auto-PR/Verify/Commit handlers to return early on ACTION(...) failures instead of discarding results. Converts previously unchecked action notifications (rate-limit, agent lifecycle, completion monitor) to result-checked flows with conditional warning emission. Removes dead-code assignments and adds failure detection for side-effecting operations.
Agentic commands – Result-based validation go/pkg/agentic/commands_forge.go, go/pkg/agentic/commands_workspace.go, go/pkg/agentic/pr.go, go/pkg/agentic/prep.go, go/pkg/agentic/scan.go	Replaces error-variable handling with explicit result.OK checks and forgeResultError(result) error extraction. Adds validation for filesystem.DeleteAll and deleteBranch outcomes; guards type assertions for milestone/label/repo results; validates git fetch and PR creation result values before consuming them.
Utility functions – error handling tightening go/pkg/agentic/auto_pr.go, go/pkg/agentic/branch_cleanup.go, go/pkg/agentic/epic.go, go/pkg/agentic/fleet_connect.go, go/pkg/agentic/flow.go, go/pkg/agentic/mirror.go, go/pkg/agentic/qa_analysis.go, go/pkg/agentic/qa_cluster.go, go/pkg/agentic/queue.go, go/pkg/lib/lib.go	Adds validation of previously-ignored return values: writeStatusResult failures, hash.Write errors, type assertion outcomes, syscall.Close errors, and config type-checking. Removes stray comment lines and improves early-return patterns on failures.
Brain package – contract updates and tests go/pkg/brain/provider_contract.go, go/pkg/brain/provider_contract_test.go, go/pkg/brain/provider_contract_example_test.go	Renames RouteDescription.Path JSON serialisation key from "path" to "route". Introduces unit tests validating zero-value behaviour, field preservation, and optional fields; adds example test demonstrating creation and printing of a route description.
Comment-only and example updates go/cmd/core-agent/commands.go, go/cmd/core-agent/main.go, go/pkg/agentic/pipeline_budget.go, go/pkg/agentic/provider_manager.go, go/pkg/agentic/review_queue.go, go/pkg/agentic/training_journal.go, go/pkg/brain/actions.go, go/pkg/brain/brain.go, go/pkg/brain/direct.go, go/pkg/monitor/monitor.go, go/pkg/runner/queue.go, go/pkg/runner/runner.go	Updates commented-out example code to assign return values to result or err variables instead of discarding with _. Removes one extraneous dead-code statement in IPC event handling.

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
12 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.7% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

❌ Patch coverage is 64.13793% with 104 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
go/pkg/agentic/dispatch.go	40.74%	8 Missing and 8 partials ⚠️
go/pkg/agentic/actions.go	0.00%	14 Missing ⚠️
go/pkg/agentcompat/agentcompat.go	77.77%	11 Missing and 1 partial ⚠️
go/pkg/agentic/forge_client.go	79.31%	9 Missing and 3 partials ⚠️
go/pkg/agentic/handlers.go	0.00%	6 Missing and 1 partial ⚠️
go/pkg/agentic/auto_pr.go	14.28%	5 Missing and 1 partial ⚠️
go/pkg/agentic/epic.go	0.00%	3 Missing and 3 partials ⚠️
go/pkg/agentic/mirror.go	0.00%	5 Missing ⚠️
go/pkg/agentic/pr.go	76.47%	2 Missing and 2 partials ⚠️
go/pkg/runner/queue.go	0.00%	4 Missing ⚠️
... and 8 more

📢 Thoughts on this report? Let us know!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud