Skip to content

Remove create-discussion feature and all associated files#522

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/remove-create-discussion-feature
Closed

Remove create-discussion feature and all associated files#522
Copilot wants to merge 1 commit intomainfrom
copilot/remove-create-discussion-feature

Conversation

Copy link
Contributor

Copilot AI commented Sep 7, 2025

This PR completely removes the create-discussion feature from the gh-aw repository as requested. The feature allowed agentic workflows to automatically create GitHub discussions as safe outputs, but it has been identified for removal.

Changes Made

Core Implementation Removal

  • JavaScript files: Removed pkg/workflow/js/create_discussion.cjs and pkg/workflow/js/create_discussion.test.cjs
  • Go integration: Removed CreateDiscussionsConfig struct, buildCreateOutputDiscussionJob function, and parseDiscussionsConfig function from the compiler
  • Embed declarations: Removed createDiscussionScript variable and embed directive from js.go

Schema and Configuration Cleanup

  • JSON schemas: Removed create-discussion entries from both main_workflow_schema.json and agent-output.json
  • Validation logic: Removed create-discussion validation from collect_ndjson_output.cjs
  • Test cases: Removed all create-discussion related test cases from Go and JavaScript test suites

Documentation Updates

  • Safe outputs documentation: Removed the entire "New Discussion Creation" section from docs/safe-outputs.md
  • Example workflows: Updated test-safe-outputs-custom-engine.md to remove create-discussion configuration and generation step
  • CLI templates: Removed create-discussion example from pkg/cli/templates/instructions.md
  • Frontmatter docs: Removed discussion-related concurrency documentation from docs/frontmatter.md

Concurrency Logic Cleanup

  • Removed functions: Eliminated isDiscussionWorkflow function and all discussion-related concurrency handling
  • Simplified logic: Cleaned up concurrency group key generation to remove discussion-specific branches
  • Test cleanup: Removed discussion workflow test cases from concurrency tests

Verification

  • ✅ All Go tests pass (go test ./...)
  • ✅ Build succeeds without errors (make build)
  • ✅ Linting passes (golangci-lint run)
  • ✅ No remaining references to create-discussion, Discussion, or CreateDiscussion found in codebase
  • ✅ CLI functionality verified (./gh-aw --help, ./gh-aw version)

The removal is complete and surgical - no other safe-output types or functionality has been affected. The codebase now operates cleanly without any create-discussion capabilities while maintaining full functionality for all other features.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • api.contoso.com
    • Triggering command: /tmp/go-build2656728407/b278/cli.test -test.testlogfile=/tmp/go-build2656728407/b278/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)
    • Triggering command: /tmp/go-build3189086267/b278/cli.test -test.testlogfile=/tmp/go-build3189086267/b278/testlog.txt -test.paniconexit0 -test.timeout=10m0s (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@pelikhan pelikhan closed this Sep 7, 2025
@pelikhan pelikhan deleted the copilot/remove-create-discussion-feature branch September 7, 2025 13:52
Copilot AI changed the title [WIP] Remove create-discussion and all associated files, documentation. Review agentic workflows and remove feature as well. Remove javascript create_discussion.cjs and tests. Remove create-discussion feature and all associated files Sep 7, 2025
Copilot AI requested a review from pelikhan September 7, 2025 14:21
@Mossaka Mossaka mentioned this pull request Feb 5, 2026
3 tasks
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