Add label support to update-discussion safe output#6968
Merged
Conversation
- Updated UpdateDiscussionsConfig struct to include Labels and AllowedLabels fields - Updated parseUpdateDiscussionsConfig to parse labels and allowed-labels - Updated generateSafeOutputsConfig to include allowed_labels in update_discussion config - Updated JSON schema to support labels and allowed-labels fields - Updated agent-output.json schema to include labels field in update_discussion type - Implemented label update logic in update_discussion.cjs using GraphQL mutations - Schema changes embedded in binary via make build - All workflows recompiled successfully Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Contributor
|
@copilot: make lint. |
Contributor
|
@copilot: Resolve merge conflicts. |
- Added Go tests for update-discussion configuration parsing with labels - Added JavaScript tests for discussion label updates - Fixed update_discussion.cjs to only query labels when needed - Updated update_runner.cjs to support labels parameter - Added canUpdateLabels to buildUpdateData function - Fixed existing tests to account for additional final query - All unit tests and JavaScript tests now pass Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update safe output to add labels from allowed list
Add label support to update-discussion safe output
Dec 19, 2025
mnkiefer
approved these changes
Dec 19, 2025
Contributor
|
@copilot allowed-labels should implicitely enable labels. |
- Fixed JavaScript formatting issues in update_discussion files - Merged origin/main to bring in latest changes - Resolved merge conflicts by recompiling all workflows - All linting checks now pass Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Contributor
Author
Done. Merged origin/main and resolved all merge conflicts by rebuilding and recompiling workflows with
Done. Linting now passes after running |
- Modified parseUpdateDiscussionsConfig to automatically set Labels when AllowedLabels is present - Updated buildUpdateDiscussionStepConfig to set GH_AW_UPDATE_LABELS env var when Labels is enabled - Added test TestUpdateDiscussionConfigAllowedLabelsImplicitlyEnablesLabels to verify behavior - Users can now specify only allowed-labels without explicitly adding labels: - All tests pass and linting is clean Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Contributor
|
@copilot: merge main and recompile. |
- Merged latest changes from origin/main - Resolved merge conflicts in pkg/workflow/compiler_yaml.go by accepting main version - Resolved merge conflicts in lock files by recompiling all workflows - All workflows recompiled successfully (117 out of 119) Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Contributor
Author
mnkiefer
approved these changes
Dec 19, 2025
mnkiefer
pushed a commit
that referenced
this pull request
Dec 19, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Plan: Add allowed-labels support to update-discussion
UpdateDiscussionsConfigstruct to includeLabelsandAllowedLabelsfieldsparseUpdateDiscussionsConfigto parselabelsandallowed-labelsconfigurationlabelsandallowed-labelsinupdate-discussiondefinitionagent-output.jsonschema to includelabelsfield inupdate_discussiontypeupdate_discussion.cjsto handle label updates via GraphQLgenerateSafeOutputsConfigto include allowed-labels in update_discussion configmake buildto rebuild with embedded schemasmake recompileto recompile all workflowsSummary
This PR adds support for updating discussion labels in the
update-discussionsafe output. All requested changes have been addressed:make fmtKey Feature: Implicit Label Enablement
When
allowed-labelsis specified, thelabelsfield is automatically enabled. This improves user experience by eliminating redundancy:Before:
After:
The implementation maintains backward compatibility - users can still explicitly specify both fields if desired.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.