Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026

update-project and create-project-status-update were processed twice: once by the dedicated project handler manager step and again by the unified handler.

Changes

Handler Architecture

  • Moved update_project and create_project_status_update from projectHandlerRegistry to handlerRegistry
  • Project handler manager now only processes create_project and copy_project
  • Added project field to both handler configs to pass project URL

JavaScript (safe_output_project_handler_manager.cjs)

const PROJECT_HANDLER_MAP = {
  create_project: "./create_project.cjs",
  copy_project: "./copy_project.cjs",
  // update_project and create_project_status_update removed
};

Go (compiler_safe_outputs_job.go)

  • Updated hasProjectHandlerManagerTypes to only check CreateProjects and CopyProjects
  • Moved permissions for UpdateProjects and CreateProjectStatusUpdates to unified handler section

Go (compiler_safe_outputs_config.go)

// Now in handlerRegistry (main handler config)
"update_project": func(cfg *SafeOutputsConfig) map[string]any { ... }
"create_project_status_update": func(cfg *SafeOutputsConfig) map[string]any { ... }

Tests

  • Updated expectations: these types now appear in GH_AW_SAFE_OUTPUTS_HANDLER_CONFIG instead of GH_AW_SAFE_OUTPUTS_PROJECT_HANDLER_CONFIG
Original prompt

update-project and create-project-status-update should be ignore the "Process Project safe outputs" steps since they handled in the JavaScript safe output unified handler.


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


Changeset

  • Type: patch
  • Description: Moved the update-project and create-project-status-update safe-output configs into the unified handler manager so workflows no longer run a separate project handler step for those types.

Ahoy! This treasure was crafted by 🏴‍☠️ Changeset Generator

Copilot AI and others added 2 commits February 1, 2026 12:14
…ect handler manager

- Update JavaScript PROJECT_HANDLER_MAP to only include create_project and copy_project
- Update Go hasProjectHandlerManagerTypes to exclude UpdateProjects and CreateProjectStatusUpdates
- Move permissions for update-project and create-project-status-update to unified handler section
- Remove obsolete tests for these types in project handler manager
- These types are now handled by the unified handler instead of the separate project step

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…handler

- Move these handlers from projectHandlerRegistry to handlerRegistry in compiler_safe_outputs_config.go
- Add project field to both handler configs so project URL is passed to handlers
- Update tests to expect these handlers in main config, not project config
- Update TestUpdateProjectWithProjectURLConfig to check handler config instead of GH_AW_PROJECT_URL
- Update TestCreateProjectStatusUpdateWithProjectURLConfig similarly
- All project-related tests now pass

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review February 1, 2026 12:21
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update project to ignore specific process steps Move update-project and create-project-status-update to unified handler Feb 1, 2026
Copilot AI requested a review from pelikhan February 1, 2026 12:26
@pelikhan pelikhan added the smoke label Feb 1, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Agent Container Tool Check

Tool Status Version
bash 5.2.21
sh not found
git 2.52.0
jq 1.7
yq 4.50.1
curl 8.5.0
gh 2.86.0
node 24.13.0
python3 3.13.11
go 1.24.12
java not found
dotnet not found

Result: 9/12 tools available ❌

Missing tools:

  • sh - sh shell not available in container
  • java - Java runtime not installed
  • dotnet - .NET runtime not installed

AI generated by Agent Container Smoke Test

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

[WIP] Remove all project-related tests from smoke-copilot workflow
[WIP] Update JavaScript front matter hash consistency algorithm
Add frontmatter hash consistency tests for Go/JavaScript implementations
Optimize Serena tool usage guidance for agentic planning
Tests: GitHub MCP ✅ | safeinputs-gh ✅ | Serena ✅ | Playwright ✅ | Tavily ✅ | file write ✅ | bash cat ✅ | discussion ✅ | build ✅
Overall status: PASS

AI generated by Smoke Codex

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Smoke Test Results

PRs Tested: #13088, #13087
Test Results:

  • ✅ GitHub MCP
  • ✅ Safe Inputs GH CLI
  • ✅ Serena MCP
  • ✅ Playwright
  • ✅ File Writing
  • ✅ Bash Tool
  • ✅ Discussion Interaction
  • ✅ Build gh-aw
  • ⚠️ Project Operations (auto-populate not working)

Status: PASS

@pelikhan @Copilot

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Smoke Test: Claude ✅

PRs Reviewed: #13088, #13087
✅ GitHub MCP
✅ Safe Inputs GH CLI
✅ Serena MCP
✅ Make Build
✅ Playwright
✅ Tavily Web Search
✅ File Writing
✅ Bash Tool
✅ Discussion Interaction

Overall Status: PASS

AI generated by Smoke Claude

@pelikhan pelikhan merged commit 91841a1 into main Feb 1, 2026
84 checks passed
@pelikhan pelikhan deleted the copilot/update-project-ignore-process-steps branch February 1, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants