Skip to content

[Nitro CLI] Merged Nitro.CommandLine.Core into Nitro.CommandLine#9079

Merged
glen-84 merged 2 commits intomainfrom
gai/merge-nitro-command-line-core
Jan 30, 2026
Merged

[Nitro CLI] Merged Nitro.CommandLine.Core into Nitro.CommandLine#9079
glen-84 merged 2 commits intomainfrom
gai/merge-nitro-command-line-core

Conversation

@glen-84
Copy link
Copy Markdown
Member

@glen-84 glen-84 commented Jan 30, 2026

Summary of the changes (Less than 80 chars)

  • [Nitro CLI] Merged Nitro.CommandLine.Core into Nitro.CommandLine.

Copilot AI review requested due to automatic review settings January 30, 2026 10:26
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 consolidates the Nitro.CommandLine.Core project into Nitro.CommandLine, eliminating the separate Core project and moving its functionality directly into the main CommandLine project.

Changes:

  • Removed the Nitro.CommandLine.Core project and all its files
  • Merged Core functionality (console helpers, extended console, exception types) into the main CommandLine project
  • Added necessary using statements to files that now reference the merged helper methods

Reviewed changes

Copilot reviewed 23 out of 27 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Nitro.CommandLine.csproj Removed project reference to CommandLine.Core
Nitro.CommandLine.slnx Removed CommandLine.Core project from solution
All.slnx Removed CommandLine.Core project from master solution
Nitro.CommandLine.Core.csproj Deleted Core project file
CommandLine.Core/Helpers/* Deleted Core helper files (merged into main project)
IExtendedConsole.cs New interface for extended console functionality
ExtendedConsole.cs New implementation of extended console
ExitException.cs New exception class for CLI exit scenarios
Glyphs.cs New file with console glyph constants (duplicate - see comments)
ConsoleHelpers.cs Added console helper extension methods from Core
ConsoleStringExtensions.cs Added AsQuestion extension method
Various command files Added using statements for ChilliCream.Nitro.CommandLine.Helpers

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

@glen-84 glen-84 merged commit d35a8e8 into main Jan 30, 2026
116 checks passed
@glen-84 glen-84 deleted the gai/merge-nitro-command-line-core branch January 30, 2026 10:48
@github-actions
Copy link
Copy Markdown
Contributor

🚀 Fusion Gateway Performance Results

Simple Composite Query

Constant Load (50 VUs)

Requests/sec Error Rate
5798.88 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
0.83ms 7.18ms 162.44ms 8.47ms 13.06ms 18.52ms

Ramping Load (0→50→500→50 VUs)

Requests/sec Error Rate
4486.59 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
0.78ms 38.64ms 248.94ms 49.42ms 110.63ms 127.02ms

Executed Query

fragment User on User {
  id
  username
  name
}

fragment Review on Review {
  id
  body
}

fragment Product on Product {
  inStock
  name
  price
  shippingEstimate
  upc
  weight
}

query TestQuery {
  topProducts(first: 5) {
    ...Product
    reviews {
      ...Review
      author {
        ...User
      }
    }
  }
}

Deep Recursion Query

Constant Load (50 VUs)

Requests/sec Error Rate
268.87 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
12.81ms 174.79ms 609.81ms 180.17ms 225.95ms 249.43ms

Ramping Load (0→50→500→50 VUs)

Requests/sec Error Rate
306.81 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
3.04ms 690.75ms 1669.37ms 687.52ms 1367.60ms 1450.46ms

Executed Query

fragment User on User {
  id
  username
  name
}

fragment Review on Review {
  id
  body
}

fragment Product on Product {
  inStock
  name
  price
  shippingEstimate
  upc
  weight
}

query TestQuery {
  users {
    ...User
    reviews {
      ...Review
      product {
        ...Product
        reviews {
          ...Review
          author {
            ...User
            reviews {
              ...Review
              product {
                ...Product
              }
            }
          }
        }
      }
    }
  }
  topProducts(first: 5) {
    ...Product
    reviews {
      ...Review
      author {
        ...User
        reviews {
          ...Review
          product {
            ...Product
          }
        }
      }
    }
  }
}

Variable Batching Throughput

Constant Load (50 VUs)

Requests/sec Error Rate
23704.88 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
0.09ms 1.70ms 48.15ms 2.06ms 3.92ms 4.80ms

Ramping Load (0→50→500→50 VUs)

Requests/sec Error Rate
18864.01 req/s 0.00%
📊 Response Time Metrics
Min Med Max Avg P90 P95
0.09ms 9.13ms 128.80ms 11.19ms 23.16ms 28.01ms

Executed Query

query TestQuery_8f7a46ce_2(
  $__fusion_1_upc: ID!
  $__fusion_2_price: Long!
  $__fusion_2_weight: Long!
) {
  productByUpc(upc: $__fusion_1_upc) {
    inStock
    shippingEstimate(weight: $__fusion_2_weight, price: $__fusion_2_price)
  }
}

Variables (5 sets batched in single request)

[
  { "__fusion_1_upc": "1", "__fusion_2_price": 899, "__fusion_2_weight": 100 },
  { "__fusion_1_upc": "2", "__fusion_2_price": 1299, "__fusion_2_weight": 1000 },
  { "__fusion_1_upc": "3", "__fusion_2_price": 15, "__fusion_2_weight": 20 },
  { "__fusion_1_upc": "4", "__fusion_2_price": 499, "__fusion_2_weight": 100 },
  { "__fusion_1_upc": "5", "__fusion_2_price": 1299, "__fusion_2_weight": 1000 }
]

No baseline data available for comparison.


Run 21513007699 • Commit 98fd511 • Fri, 30 Jan 2026 10:59:33 GMT

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