fix(ci): lint + checkout-eyrie fallback + marketplace sync (#83)

* fix(ci): lint + checkout-eyrie fallback + marketplace sync

* style: fix gofumpt formatting in chat_scrollbar.go

* fix(ci): add sync.Mutex to fix race in executeToolCalls

Author: Patel230

.github/actions/checkout-eyrie/action.yml

@@ -29,7 +29,17 @@ runs:
             # (e.g. by a squash-merge). A depth-1 clone can't check
             # out older commits and fails with "unable to read tree".
             git clone "https://github.com/GrayCodeAI/${repo}.git" "$dest"
-            (cd "$dest" && git checkout --quiet "$commit")
+            if ! (cd "$dest" && git checkout --quiet "$commit" 2>/dev/null); then
+              echo "Submodule commit $commit not reachable, falling back to ref"
+              ref="${{ inputs.ref }}"
+              if [ "$ref" = "main" ]; then
+                rm -rf "$dest"
+                git clone --depth=1 --branch main \
+                  "https://github.com/GrayCodeAI/${repo}.git" "$dest"
+              else
+                (cd "$dest" && git fetch --depth=1 "origin" "$ref" && git checkout --quiet "$ref")
+              fi
+            fi
           else
             ref="${{ inputs.ref }}"
             # Fall back to main if the branch doesn't exist on the dependency repo.

cmd/chat.go

@@ -578,7 +578,7 @@ func runChat() error {
 	}
 	systemPrompt := promptRes.text
 	settings := settingsRes.settings
-	m, err := newChatModelWithRegistry(ref, systemPrompt, settings, registryRes.registry)
+	m, err := newChatModel(ref, systemPrompt, settings)
 	if err != nil {
 		return err
 	}

cmd/chat_scrollbar.go

@@ -17,12 +17,8 @@ const (
 	scrollbarBottomGlyph = "╵" // cap at the very bottom of the track
 )
 
-var (
-	// scrollbarThumbStyle — brand orange thumb so it pops.
-	scrollbarThumbStyle = lipgloss.NewStyle().Foreground(hawkColor)
-	// scrollbarTrackStyle — very dim grey track so it doesn't distract.
-	scrollbarTrackStyle = lipgloss.NewStyle().Foreground(lipgloss.Color("#3A3A3A"))
-)
+// scrollbarThumbStyle — brand orange thumb so it pops.
+var scrollbarThumbStyle = lipgloss.NewStyle().Foreground(hawkColor)
 
 // chatHasOverflow reports whether chat content exceeds the viewport height.
 func (m chatModel) chatHasOverflow() bool {

cmd/chat_stream.go

@@ -26,12 +26,6 @@ func (m *chatModel) startPromptCommand(display, prompt string) (tea.Model, tea.C
 	return m, nil
 }
 
-// dispatchStreamEvent maps one engine event to TUI messages. Returns true when
-// the pump should stop (error or done).
-func dispatchStreamEvent(ref *progRef, ev engine.StreamEvent) bool {
-	return dispatchStreamEventWithFlush(ref, ev, nil)
-}
-
 func dispatchStreamEventWithFlush(ref *progRef, ev engine.StreamEvent, flush func()) bool {
 	if ev.Type != "content" && flush != nil {
 		flush()

external/sight

@@ -161,10 +161,6 @@ func ToolSummary(name string, args map[string]interface{}) string {
 	return name
 }
 
-func boolPtr(v bool) *bool {
-	return &v
-}
-
 func pathArgument(args map[string]interface{}) (string, bool) {
 	if p, ok := args["path"].(string); ok && p != "" {
 		return p, true

internal/engine/safety/permission.go

@@ -25,18 +25,6 @@ type toolExecResult struct {
 	isErr  bool
 }
 
-// classifyToolCalls splits tool calls into concurrent (read-only) and sequential (write) batches.
-func classifyToolCalls(calls []types.ToolCall) (concurrent, sequential []types.ToolCall) {
-	for _, tc := range calls {
-		if tool.IsReadOnly(tc.Name) {
-			concurrent = append(concurrent, tc)
-		} else {
-			sequential = append(sequential, tc)
-		}
-	}
-	return
-}
-
 // filePathArgKeys is the list of argument names that are conventionally
 // file paths. Tools with non-standard names silently fall through and
 // extractTargets returns an empty list. For a more robust extraction, see
@@ -203,6 +191,7 @@ func (s *Session) executeToolCalls(ctx context.Context, toolCalls []types.ToolCa
 	readOnlySem := make(chan struct{}, maxConcurrentReadOnlyToolCalls)
 	networkSem := make(chan struct{}, maxConcurrentNetworkReadOnlyToolCalls)
 	var wg sync.WaitGroup
+	var mu sync.Mutex
 
 	for _, item := range concurrentCalls {
 		wg.Add(1)
@@ -214,13 +203,17 @@ func (s *Session) executeToolCalls(ctx context.Context, toolCalls []types.ToolCa
 				networkSem <- struct{}{}
 				defer func() { <-networkSem }()
 			}
+			mu.Lock()
 			results[item.index] = s.executeSingleTool(ctx, item.tc, ch, turnCount, intentText)
+			mu.Unlock()
 		}(item)
 	}
 	wg.Wait()
 
 	for _, item := range sequentialCalls {
+		mu.Lock()
 		results[item.index] = s.executeSingleTool(ctx, item.tc, ch, turnCount, intentText)
+		mu.Unlock()
 	}
 
 	return results

internal/engine/stream_tool_exec.go

@@ -9,9 +9,9 @@ Refines raw ideas into sharp, actionable concepts worth building through structu
 
 ## How It Works
 
-1.  **Understand & Expand (Divergent):** Restate the idea, ask sharpening questions, and generate variations.
-2.  **Evaluate & Converge:** Cluster ideas, stress-test them, and surface hidden assumptions.
-3.  **Sharpen & Ship:** Produce a concrete markdown one-pager moving work forward.
+1. **Understand & Expand (Divergent):** Restate the idea, ask sharpening questions, and generate variations.
+2. **Evaluate & Converge:** Cluster ideas, stress-test them, and surface hidden assumptions.
+3. **Sharpen & Ship:** Produce a concrete markdown one-pager moving work forward.
 
 ## Usage
 

internal/plugin/bundled_skills/idea-refine/SKILL.md

@@ -87,6 +87,7 @@ One thing I want to push back on: you said variation 1 feels "necessary but bori
 
 **Skill produces:**
 
+<!-- markdownlint-disable MD025 -->
 # ReOrder: Keep Your Regulars Ordering Direct
 
 ## Problem Statement

internal/plugin/bundled_skills/idea-refine/examples.md

@@ -316,7 +316,6 @@ npx lhci autorun
 
 For detailed performance checklists, optimization commands, and anti-pattern reference, see `references/performance-checklist.md`.
 
-
 ## Common Rationalizations
 
 | Rationalization | Reality |