Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

Plan: Remove update-project and create-project-status-update safe outputs

  • Understand the current implementation
    • Located all files referencing UpdateProjects and CreateProjectStatusUpdates
    • Identified project_safe_outputs.go as the auto-configuration file
    • Reviewed test files to understand expected behavior
  • Remove auto-configuration of these safe outputs
    • Removed auto-configuration logic from project_safe_outputs.go
    • Updated tests in project_safe_outputs_test.go
  • Remove safe output field definitions
    • Removed UpdateProjects field from SafeOutputsConfig in compiler_types.go
    • Removed CreateProjectStatusUpdates field from SafeOutputsConfig
    • Removed parsing functions in safe_outputs_config.go
  • Remove job building and handlers
    • Updated compiler_safe_outputs_job.go to remove references
    • Updated compiler_safe_outputs_steps.go to remove references
    • Removed from compiler_safe_outputs_config.go handler registries
    • Created project_types.go for shared ProjectView and ProjectFieldDefinition types
  • Clean up helper and reflection utilities
    • Removed mappings from safe_outputs_config_helpers_reflection.go
    • Updated safe_outputs_config_generation.go
    • Updated imports.go to remove merge logic
  • Remove from JSON schema
    • Removed from pkg/parser/schemas/main_workflow_schema.json
    • Removed from .github/aw/main_workflow_schema.json
    • Updated project field descriptions
  • Clean up test files
    • Removed update_project_test.go
    • Removed update_project_token_test.go
    • Removed update_project_handler_config_test.go
    • Removed create_project_status_update_handler_config_test.go
    • Updated safe_outputs_test.go
    • Updated safe_outputs_import_test.go
    • Updated safe_outputs_integration_test.go
    • Updated compiler_safe_outputs_steps_test.go
  • Remove JavaScript handler files
    • Removed actions/setup/js/update_project.cjs
    • Removed actions/setup/js/update_project.test.cjs
  • Remove workflow examples
    • Removed test-project-url-default.md
    • Updated shared/campaign.md
    • Removed security-alert-burndown.md
  • Run tests and validation
    • Run unit tests - All pass
    • Run make recompile - Successfully compiled all workflows
    • Recompiled lock files and removed orphaned ones
Original prompt

Remove update-project and create-project-update-status from project related safe outputs.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 31, 2026 14:50
…safe outputs

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

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI and others added 2 commits January 31, 2026 14:56
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
@pelikhan pelikhan closed this Jan 31, 2026
@pelikhan
Copy link
Contributor

Derailled

Copilot AI requested a review from mnkiefer January 31, 2026 15:06
Copilot stopped work on behalf of mnkiefer due to an error January 31, 2026 15:06
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