Skip to content

[Nitro CLI] Add --exclude-by-tag option to compose command#9110

Merged
tobias-tengler merged 2 commits into
ChilliCream:mainfrom
AllenInstitute:exclude-tag
Feb 13, 2026
Merged

[Nitro CLI] Add --exclude-by-tag option to compose command#9110
tobias-tengler merged 2 commits into
ChilliCream:mainfrom
AllenInstitute:exclude-tag

Conversation

@madiganz
Copy link
Copy Markdown
Contributor

Previously exclude-by-tag could only set on the settings command, which can only be run after an archive file has already been created. This leads to a problem if all sources need to be merged without a specific tag. This change updates the nitro compose options to support excluding by tag.

Copilot AI review requested due to automatic review settings February 13, 2026 05:37
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Feb 13, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for the --exclude-by-tag option to the fusion compose command, enabling users to exclude schema elements by tag during composition without needing to create an archive first and then use the settings command.

Changes:

  • Added a new ExcludeTagListOption class for handling the --exclude-by-tag command-line option
  • Modified FusionComposeCommand to accept and process the exclude-by-tag option during composition
  • Added comprehensive test coverage with test resources demonstrating tag exclusion functionality

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ExcludeTagListOption.cs New option class defining the --exclude-by-tag command-line argument for accepting a list of tags to exclude
FusionComposeCommand.cs Updated compose command to retrieve, pass through, and apply the exclude-by-tag option throughout all execution paths (standard, watch mode, and file watching)
FusionComposeCommandTests.cs Added test case Compose_Valid_ExcludeTag to verify the exclude-by-tag functionality works correctly
source-schema-1.graphqls Test resource containing a GraphQL schema with fields tagged with "exclude-1" and "exclude-2"
source-schema-1-settings.json Configuration for the first test schema
source-schema-2.graphqls Test resource containing a second GraphQL schema without tagged fields
source-schema-2-settings.json Configuration for the second test schema
composite-schema.graphqls Expected result showing that fields with "exclude-1" and "exclude-2" tags are excluded from composition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/Nitro/CommandLine/src/CommandLine/Commands/Fusion/FusionComposeCommand.cs Outdated
@tobias-tengler tobias-tengler changed the title Add exclude-by-tag option to nitro compose command [Nitro CLI] Add exclude-by-tag option to compose command Feb 13, 2026
@tobias-tengler tobias-tengler changed the title [Nitro CLI] Add exclude-by-tag option to compose command [Nitro CLI] Add --exclude-by-tag option to compose command Feb 13, 2026
@tobias-tengler tobias-tengler merged commit d6466f3 into ChilliCream:main Feb 13, 2026
93 of 115 checks passed
@madiganz madiganz deleted the exclude-tag branch February 13, 2026 21:02
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.

4 participants