Address review feedback and implement group ordering functionality

- Move groupOrder props to base interfaces in ui.tsx (removed duplicate props)
- Simplify SelectInput sorting logic to use function-based approach
- Fix group order extraction in app-management-client to use original templates
- Add comprehensive tests for group ordering functionality
- Update snapshots to reflect new groupOrder behavior
- Fix ESLint comment positioning errors

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: jenstanicak

packages/app/src/cli/utilities/developer-platform-client/app-management-client.test.ts

@@ -212,7 +212,8 @@ describe('templateSpecifications', () => {
     const templates: GatedExtensionTemplate[] = [
       {...templateWithoutRules, group: 'GroupA'},
       {...allowedTemplate, group: 'GroupB'},
-      {...templateWithoutRules, identifier: 'template3', group: 'GroupA'}, // Same group as first
+      // Same group as first
+      {...templateWithoutRules, identifier: 'template3', group: 'GroupA'},
       {...allowedTemplate, identifier: 'template4', group: 'GroupC'},
     ]
     const mockedFetch = vi.fn().mockResolvedValueOnce(Response.json(templates))

packages/cli-kit/src/private/node/ui/components/SelectInput.test.tsx

@@ -248,19 +248,21 @@ describe('SelectInput', async () => {
     const renderInstance = render(<SelectInput items={items} onChange={onChange} groupOrder={groupOrder} />)
 
     expect(renderInstance.lastFrame()).toMatchInlineSnapshot(`
-      "   [1mGroupC[22m
-         [36m>[39m  [36mfifth[39m
+      "   GroupC
+         >  fifth
             sixth
 
-         [1mGroupB[22m
+         GroupB
             third
             fourth
 
-         [1mGroupA[22m
+         GroupA
             first
             second
 
-         [2mPress ↑↓ arrows to select, enter to confirm.[22m"
+
+
+         Press ↑↓ arrows to select, enter to confirm."
     `)
     expect(onChange).not.toHaveBeenCalled()
   })
@@ -381,16 +383,16 @@ describe('SelectInput', async () => {
     const renderInstance = render(<SelectInput items={items} onChange={() => {}} availableLines={10} />)
 
     expect(renderInstance.lastFrame()).toMatchInlineSnapshot(`
-      "   Automations                                                                  
-         >  fifth                                                                     
-            sixth                                                                     
-                                                                                      
-         Merchant Admin                                                               
-            eighth                                                                    
-            ninth                                                                     
+      "   Other                                                                        
+         >  first                                                                     
+            second                                                                    
+            third                                                                     
+            fourth                                                                    
+                                                                                      
+         Automations                                                                  
+            fifth                                                                     
+            sixth                                                                     
                                                                                       
-         Other                                                                        
-            first                                                                     
 
          Press ↑↓ arrows to select, enter to confirm."
     `)
@@ -403,16 +405,16 @@ describe('SelectInput', async () => {
     await sendInputAndWaitForChange(renderInstance, ARROW_DOWN)
 
     expect(renderInstance.lastFrame()).toMatchInlineSnapshot(`
-      "   Automations                                                                  
-            sixth                                                                     
+      "   Other                                                                        
+            second                                                                    
+            third                                                                     
+            fourth                                                                    
                                                                                       
-         [1mMerchant Admin[22m                                                              [46m [49m
-            eighth                                                                   [46m [49m
-            ninth                                                                    [46m [49m
+         [1mAutomations[22m                                                                 [46m [49m
+            fifth                                                                    [100m [49m
+         [36m>[39m  [36msixth[39m                                                                    [100m [49m
                                                                                       
          Other                                                                        
-            first                                                                     
-         >  second                                                                    
 
          Press ↑↓ arrows to select, enter to confirm."
     `)

packages/cli-kit/src/public/node/ui.tsx

@@ -256,21 +256,21 @@ export interface RenderSelectPromptOptions<T> extends Omit<SelectPromptProps<T>,
  *    ┃  • integrated-demand-ext
  *    ┃  • order-discount
  *
- *    Automations
- *    >  fifth
- *       sixth
- *
- *    Merchant Admin
- *       eighth
- *       ninth
- *
  *    Other
- *       first
+ *    >  first
  *       second
  *       third (limit reached)
  *       fourth
+ *
+ *    Automations
+ *       fifth
+ *       sixth
+ *
+ *    Other
  *       seventh
- *       tenth
+ *
+ *    Merchant Admin
+ *       eighth
  *
  *    Press ↑↓ arrows to select, enter to confirm.
  *