…326)

* feat(phase1): Implement real Go build/test steps in squad workflows #322

Replace TODO placeholders in squad CI/preview/release workflows with:
- Go mod verify, go vet, go test with coverage, race detector
- Node.js web UI build (Node 22 with npm cache)
- Binary build and integration test for end-to-end validation
- Release creation with gh CLI (main) and pre-release (insider)

All workflows now mirror go-ci.yml proven build/test pattern:
- squad-ci.yml: Runs on dev/preview/main/insider branches and PRs
- squad-preview.yml: Validates on preview branch pushes
- squad-release.yml: Runs tests and creates release on main
- squad-insider-release.yml: Runs tests and creates pre-release on insider

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: Add failure detection and artifact capture infrastructure #322

Phase 2: Implement genuine failure-handling for evaluation runs

- Add failures/handler.go with CaptureFailure method
- Capture stderr/stdout (10KB truncation), exit code, failed validators
- Extract error patterns via regex (timeout, OOM, permission, connection, panic, exception)
- Generate human-readable triage summary in Markdown with actionable recommendations
- Extend RunResult.FailureArtifacts for diagnostic data in results JSON
- Add comprehensive test coverage (handler_test.go)
- All tests passing, 79.1% coverage maintained

Next: Wire handler into EvalRunner, add --auto-file-issue flag, display triage in CLI

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix(ci): restore Go 1.26, fix golangci-lint to exclude web/node_modules #322

- Revert go.mod back to 1.26 (dependency azure-dev/cli/azd requires >= 1.26)
- Restore go-version: 1.26 in go-ci.yml and squad-ci.yml
- Add linters.exclusions.paths to .golangci.yml to skip web/node_modules
  and web/dist (uses v2 schema: linters.exclusions.paths)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix(ci): update Squad CI lint to use golangci-lint-action@v7 v2.10.1 #322

v4 with 'latest' resolves to v1.64.8 (built with Go 1.24) which
cannot lint Go 1.26 projects. Upgrade to action@v7 with pinned v2.10.1
(built with Go 1.26) to match go-ci.yml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address review comments in failure handler

- Add exitCode parameter to CaptureFailure and populate ExitCode field
- Sort FailedGraders for deterministic output
- Sort extractErrorPatterns result for deterministic output
- Fix contains() helper to use strings.Contains
- Fix TestTruncate: correct wantLen and assert actual length
- Fix TestExtractErrorPatterns: assert specific expected patterns
- Update TestCaptureFailure: pass exit code, assert ExitCode field
- Drop unused contains() helper (now using strings.Contains directly)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: replace error-suppressing release creation with idempotent check

Use 'gh release view' to check if a release exists before creating,
so genuine failures (permission errors, network issues) surface as
workflow failures rather than being swallowed by '|| echo'.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address all 14 new review comments from PR #324

- handler.go: use time.Now().UTC() for cross-machine consistency
- handler.go: filter empty strings from extractErrorPatterns matches
- handler.go: fix truncate to enforce hard maxLen (prefix len = maxLen - suffix len)
- handler_test.go: update TestTruncate wantLen to maxLen (30) after hard-limit fix
- squad-preview/release/insider-release.yml: add favicon.svg to web/dist stub
- squad-release/insider-release.yml: change 'passed' to 'completed' in integration test
  message since exit code 1 (eval failures) is treated as non-fatal
- go-ci.yml lint job: fix go-version indentation to 10 spaces under with:
- squad-ci.yml lint job: fix go-version indentation to 10 spaces under with:

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address 3 new lint/review issues after second review pass

- handler_test.go: remove unused truncSuffix const (fixes lint)
- handler.go: use regexp.MustCompile for hard-coded patterns to surface
  typos at runtime instead of silently ignoring compile errors
- squad-ci.yml: add lfs: true to lint job checkout to match build job
- .golangci.yml exclusions.paths is valid v2 config (config verify passes)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: move exclusions.paths to top-level in golangci.yml v2 config

In golangci-lint v2, path exclusions belong under the top-level
exclusions key, not under linters.exclusions. Moving them ensures
they are applied correctly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: complete issue 322 reliability loops and triage automation #322

- add CLI auto issue filing and triage highlights for failed runs
- wire failure artifact capture into runner execution paths
- add weekly regression loop workflow with baseline artifact comparison and issue upsert
- harden auto-merge workflow with trusted-author + label gates
- document new run flag and CI workflows, and fix lint package resolution

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: remove invalid exclusions from golangci.yml

golangci-lint config verify was failing because the top-level 'exclusions'
property is not valid in the current schema. Removed this section as the web
directory is not a Go package and does not need explicit exclusion.

Fixes: Go Build and Test CI failure due to golangci-lint config verification.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>