Skip to content

Hide PR artifacts from manager assignments#429

Merged
nathanwhit merged 1 commit into
mainfrom
codex/hide-pr-artifacts-manager
Jun 11, 2026
Merged

Hide PR artifacts from manager assignments#429
nathanwhit merged 1 commit into
mainfrom
codex/hide-pr-artifacts-manager

Conversation

@nathanwhit

Copy link
Copy Markdown
Owner

Summary

  • stop treating github_pull_request artifacts as manager-visible assignment artifacts
  • rely on first-class PR rows for active PRs, so merged/closed PR outputs do not linger as Available artifact rows
  • keep non-PR artifacts such as benchmark outputs visible

Validation

  • go test ./internal/orchestrator -run TestBuildTaskAssignmentsDisplayRowsHideHistoricalFailures
  • go test ./internal/eventstore -run TestBuildManagerSummaries
  • go test ./internal/orchestrator ./internal/eventstore
  • npm run test -- AssignmentBoard
  • npm run build

@nathanwhit nathanwhit force-pushed the codex/hide-pr-artifacts-manager branch from 78370b2 to fd57cbc Compare June 11, 2026 04:49
@nathanwhit nathanwhit merged commit f0b092c into main Jun 11, 2026
2 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