Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026

After moving update_project, create_project_status_update, and create_project to the unified handler manager, the old handler manager was still marking them as standalone steps, causing messages to be skipped instead of processed.

Changes

  • Removed project types from STANDALONE_STEP_TYPES in safe_output_handler_manager.cjs

    • Removed create_project, update_project, create_project_status_update from the set
    • These are now handled exclusively by the unified handler manager
  • Removed PROJECT_RELATED_TYPES constant

    • No longer needed since these types aren't standalone
    • Simplified skip reason logic to single path
  • Updated tests to use remaining standalone types (create_agent_session, upload_asset)

The unified handler manager (safe_output_unified_handler_manager.cjs) already has these types correctly configured in PROJECT_HANDLER_MAP and was updated separately per the changeset documentation.

Original prompt

Fix Skipped (project handler step) for types: update_project, create_project_status_update. See: https://github.com/githubnext/gh-aw/actions/runs/21569768903/job/62147021861


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

Copilot AI and others added 2 commits February 1, 2026 20:47
- Remove create_project, update_project, and create_project_status_update from STANDALONE_STEP_TYPES
- Remove PROJECT_RELATED_TYPES constant (no longer needed)
- Update tests to reflect the changes
- These types are now handled by the unified handler manager, not separate steps

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix skipped project handler step for update and create project status Fix project handler types incorrectly marked as standalone steps Feb 1, 2026
Copilot AI requested a review from mnkiefer February 1, 2026 20:54
@mnkiefer mnkiefer marked this pull request as ready for review February 1, 2026 20:56
@mnkiefer
Copy link
Contributor

mnkiefer commented Feb 1, 2026

@copilot Recompile workflows

Copy link
Contributor Author

Copilot AI commented Feb 1, 2026

@copilot Recompile workflows

Recompiled all workflows successfully. No changes to .lock.yml files were needed since the modifications to safe_output_handler_manager.cjs are runtime JavaScript files that execute during workflow runs, not during compilation.

Copilot AI requested a review from mnkiefer February 1, 2026 21:05
@mnkiefer mnkiefer merged commit 2c1d9de into main Feb 1, 2026
127 checks passed
@mnkiefer mnkiefer deleted the copilot/fix-skipped-project-handler-step branch February 1, 2026 21:08
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.

2 participants