Skip to content

[WIP] Split update and create project status update#12961

Closed
Copilot wants to merge 5 commits intomainfrom
copilot/split-update-and-create-project
Closed

[WIP] Split update and create project status update#12961
Copilot wants to merge 5 commits intomainfrom
copilot/split-update-and-create-project

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

Complete: Split update-project and create-project-status-update from project handler manager

Goal: Move update_project and create_project_status_update handlers from the project-specific handler manager to the regular safe outputs handler manager. ✅

Summary of Changes:

This PR successfully splits update-project and create-project-status-update handlers from the project-specific handler manager and moves them to the regular safe outputs handler manager. These handlers now run with regular GitHub tokens instead of requiring the specialized GH_AW_PROJECT_GITHUB_TOKEN.

Key Changes:

  • Move update_project and create_project_status_update to regular handler manager (HANDLER_MAP)
  • Remove them from project handler manager (PROJECT_HANDLER_MAP)
  • Update Go code to remove references to project handler manager for these types
  • Update configuration generation logic in compiler_safe_outputs_config.go
  • Update step building logic in compiler_safe_outputs_job.go and compiler_safe_outputs_steps.go
  • Update all tests to reflect new architecture
  • Run tests to verify changes - All tests passing ✅
  • Recompile all 147 workflows successfully ✅
  • Format all code ✅

Architecture After Changes:

Project Handler Manager (requires GH_AW_PROJECT_GITHUB_TOKEN):

  • create_project
  • copy_project

Regular Handler Manager (uses standard GitHub token):

  • All existing handlers (create_issue, update_issue, etc.)
  • NEW: update_project
  • NEW: create_project_status_update

Benefits:

  • Simplified token management for update-project and create-project-status-update operations
  • These handlers can now use regular GitHub permissions instead of requiring Projects v2 PAT
  • Maintains backward compatibility with existing workflows
  • Proper separation: only create_project and copy_project need special project tokens

Files Modified:

  • actions/setup/js/safe_output_handler_manager.cjs - Added update_project and create_project_status_update
  • actions/setup/js/safe_output_project_handler_manager.cjs - Removed handlers, updated comments
  • pkg/workflow/compiler_safe_outputs_config.go - Moved handlers between registries
  • pkg/workflow/compiler_safe_outputs_job.go - Updated permissions and step logic
  • pkg/workflow/compiler_safe_outputs_steps.go - Updated token selection logic
  • pkg/workflow/create_project_status_update_handler_config_test.go - Updated test expectations
  • pkg/workflow/update_project_handler_config_test.go - Updated test expectations
  • pkg/workflow/compiler_safe_outputs_steps_test.go - Updated test cases
  • All 147 workflow lock files recompiled
Original prompt

Split the update-project and create-project-status-update and let them run with regular safe outputs


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 4 commits January 31, 2026 15:46
…t-status-update

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
…outputs

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
…e handlers

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
…us-update handlers

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
@pelikhan pelikhan closed this Jan 31, 2026
Copilot AI requested a review from mnkiefer January 31, 2026 15:54
Copilot stopped work on behalf of mnkiefer due to an error January 31, 2026 15:54
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.

3 participants