Skip to content

fix(agents): convert tools frontmatter from array to comma-separated string #2528

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
MeGaurav4 wants to merge 1 commit into from
+67 −67
Closed

fix(agents): convert tools frontmatter from array to comma-separated string #2528

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion agents/a11y-architect.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: a11y-architect
description: Accessibility Architect specializing in WCAG 2.2 compliance for Web and Native platforms. Use PROACTIVELY when designing UI components, establishing design systems, or auditing code for inclusive user experiences.
model: sonnet
tools: ["Read", "Write", "Edit", "Grep", "Glob"]
tools: Read, Write, Edit, Grep, Glob

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

The scalar format breaks repository consumers that still require arrays.

  • agents/a11y-architect.md#L5-L5: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/agent-evaluator.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/architect.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/build-error-resolver.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/kotlin-build-resolver.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/kotlin-reviewer.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/loop-operator.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/marketing-agent.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/mle-reviewer.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/network-architect.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/network-config-reviewer.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
  • agents/network-troubleshooter.md#L4-L4: normalize the scalar in downstream metadata consumers or preserve an array-compatible boundary.
📍 Affects 12 files
  • agents/a11y-architect.md#L5-L5 (this comment)
  • agents/agent-evaluator.md#L4-L4
  • agents/architect.md#L4-L4
  • agents/build-error-resolver.md#L4-L4
  • agents/kotlin-build-resolver.md#L4-L4
  • agents/kotlin-reviewer.md#L4-L4
  • agents/loop-operator.md#L4-L4
  • agents/marketing-agent.md#L4-L4
  • agents/mle-reviewer.md#L4-L4
  • agents/network-architect.md#L4-L4
  • agents/network-config-reviewer.md#L4-L4
  • agents/network-troubleshooter.md#L4-L4
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/a11y-architect.md` at line 5, The metadata format currently exposes a
scalar where downstream repository consumers require an array. Update the
metadata boundary represented by the tools declarations so consumers
consistently receive an array-compatible value, either by normalizing the scalar
downstream or preserving the array form at the source. Apply the same correction
at agents/a11y-architect.md:5-5, agents/agent-evaluator.md:4-4,
agents/architect.md:4-4, agents/build-error-resolver.md:4-4,
agents/kotlin-build-resolver.md:4-4, agents/kotlin-reviewer.md:4-4,
agents/loop-operator.md:4-4, agents/marketing-agent.md:4-4,
agents/mle-reviewer.md:4-4, agents/network-architect.md:4-4,
agents/network-config-reviewer.md:4-4, and agents/network-troubleshooter.md:4-4.

---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/agent-evaluator.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: agent-evaluator
description: Evaluates agent output against 5-axis quality rubric (accuracy, completeness, clarity, actionability, conciseness). Use after any non-trivial task when the user wants a quality assessment, or when the agent-self-evaluation skill is active. Produces structured scorecard with evidence and improvement suggestions.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔒 Security & Privacy | 🟠 Major | 🏗️ Heavy lift

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## agents/agent-evaluator.md (first 80 lines)\n'
nl -ba agents/agent-evaluator.md | sed -n '1,80p'

printf '\n## files in agents/ and nearby configs mentioning Bash or tools\n'
rg -n --hidden --glob 'agents/**/*.md' --glob 'skills/**/*.md' --glob 'commands/**/*.md' --glob 'rules/**/*.md' -e '^tools:' -e 'Bash' -e 'allowlist' -e 'permission' -e 'read-only' -e 'destructive' -e 'prompt-injection' -e 'frontmatter' .

Repository: affaan-m/ECC

Length of output: 237

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## agents/agent-evaluator.md (first 120 lines)\n'
sed -n '1,120p' agents/agent-evaluator.md | cat -n

printf '\n## agent/tool/permission references\n'
rg -n --hidden --glob 'agents/**/*.md' --glob 'skills/**/*.md' --glob 'commands/**/*.md' --glob 'rules/**/*.md' \
  -e '^tools:' -e '^model:' -e 'Bash' -e 'allowlist' -e 'permission' -e 'read-only' -e 'destructive' -e 'prompt-injection' -e 'frontmatter' .

Repository: affaan-m/ECC

Length of output: 47330

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## agent harness / hook references for tool enforcement\n'
rg -n --hidden \
  -e 'allowedTools' \
  -e 'tools:' \
  -e 'PreToolUse' \
  -e 'Bash Tool Constraints' \
  -e 'permission' \
  -e 'read-only' \
  -e 'hook' \
  skills/agents commands rules agents | sed -n '1,220p'

printf '\n## relevant docs: agent-harness-construction and hook integration\n'
for f in skills/agent-harness-construction/SKILL.md skills/agent-self-evaluation/references/hook-integration.md skills/safety-guard/SKILL.md; do
  if [ -f "$f" ]; then
    echo "--- $f ---"
    sed -n '1,220p' "$f" | cat -n
  fi
done

Repository: affaan-m/ECC

Length of output: 23727

🏁 Script executed:

#!/bin/bash
set -euo pipefail

for f in rules/common/hooks.md skills/agent-harness-construction/SKILL.md skills/agent-self-evaluation/references/hook-integration.md; do
  if [ -f "$f" ]; then
    echo "--- $f ---"
    sed -n '1,220p' "$f" | cat -n
  fi
done

printf '\n## direct references to agent-evaluator / runtime allowlists / hooks.json\n'
rg -n --hidden \
  -e 'agent-evaluator' \
  -e 'allowedTools' \
  -e 'hooks/hooks.json' \
  -e 'PreToolUse' \
  -e 'PostToolUse' \
  -e 'dangerously-skip-permissions' \
  -e 'runtime permission' \
  -e 'tools are granted' \
  rules skills agents commands hooks . | sed -n '1,240p'

Repository: affaan-m/ECC

Length of output: 37062

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## agent-evaluator files in .kiro and repo\n'
find . -path '*/agent-evaluator*' -o -path '*/agent-evaluator.*' | sed -n '1,120p'

printf '\n## agent registries / loaders mentioning agents directory or .kiro/agents\n'
rg -n --hidden \
  -e '\.kiro/agents' \
  -e 'agents/' \
  -e 'allowedTools' \
  -e 'agent-evaluator' \
  -e 'load.*agent' \
  -e 'frontmatter' \
  -e 'yaml' \
  commands scripts hooks skills . | sed -n '1,220p'

Repository: affaan-m/ECC

Length of output: 19331

Remove Bash from this read-only agent, or enforce it in runtime policy — the prompt-only read-only note at agents/agent-evaluator.md doesn’t constrain the granted tool. If shell access is required, scope it through the runtime allowlist/hook layer; otherwise drop Bash from tools.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/agent-evaluator.md` at line 4, Remove Bash from the tools declaration
for the read-only agent in agents/agent-evaluator.md, unless runtime policy
explicitly enforces a restricted shell allowlist; do not rely solely on the
prompt’s read-only note to constrain access.

Source: Path instructions

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/architect.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: architect
description: Software architecture specialist for system design, scalability, and technical decision-making. Use PROACTIVELY when planning new features, refactoring large systems, or making architectural decisions.
tools: ["Read", "Grep", "Glob"]
tools: Read, Grep, Glob
model: opus
---

Expand Down
2 changes: 1 addition & 1 deletion agents/build-error-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: build-error-resolver
description: Build and TypeScript error resolution specialist. Use PROACTIVELY when build fails or type errors occur. Fixes build/type errors only with minimal diffs, no architectural edits. Focuses on getting the build green quickly.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/chief-of-staff.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: chief-of-staff
description: Personal communication chief of staff that triages email, Slack, LINE, and Messenger. Classifies messages into 4 tiers (skip/info_only/meeting_info/action_required), generates draft replies, and enforces post-send follow-through via hooks. Use when managing multi-channel communication workflows.
tools: ["Read", "Grep", "Glob", "Bash", "Edit", "Write"]
tools: Read, Grep, Glob, Bash, Edit, Write

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== agent files ==\n'
git ls-files 'agents/*.md' | sed -n '1,120p'

printf '\n== search for frontmatter tools handling ==\n'
rg -n --hidden --glob '!node_modules' --glob '!dist' --glob '!build' 'tools[:=]|frontmatter|gray-matter|yaml|matter|agent.*tools|comma-separated|string' .

printf '\n== outline likely loaders (if any) ==\n'
fd -t f -a '.*(agent|frontmatter|yaml|md).*' . | sed -n '1,200p'

Repository: affaan-m/ECC

Length of output: 50371

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== agent frontmatter samples ==\n'
for f in agents/chief-of-staff.md agents/code-architect.md agents/code-explorer.md agents/code-reviewer.md agents/code-simplifier.md agents/opensource-forker.md agents/opensource-packager.md agents/opensource-sanitizer.md agents/performance-optimizer.md agents/php-reviewer.md agents/planner.md; do
  echo "--- $f"
  sed -n '1,12p' "$f"
done

printf '\n== files that parse agent markdown/frontmatter ==\n'
rg -n --hidden --glob '!node_modules' --glob '!dist' --glob '!build' --glob '!*test*' --glob '!tests/**' \
  'agents/.*\.md|frontmatter|gray-matter|matter|yaml-front|parse.*agent|load.*agent|tools:' .

Repository: affaan-m/ECC

Length of output: 50371

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== scripts/lib/agent-compress.js ==\n'
sed -n '1,260p' scripts/lib/agent-compress.js

printf '\n== scripts/dashboard-web.js relevant frontmatter parsing ==\n'
sed -n '1,120p' scripts/dashboard-web.js

printf '\n== scripts/ci/validate-agents.js ==\n'
sed -n '1,140p' scripts/ci/validate-agents.js

Repository: affaan-m/ECC

Length of output: 25236

Keep tools as a YAML array in all listed agent frontmatter blocks. The catalog/dashboard loaders only preserve tools when it parses as an array; scalar values become [], so these agents will render with empty tool lists. Same issue applies to the other listed files.

📍 Affects 11 files
  • agents/chief-of-staff.md#L4-L4 (this comment)
  • agents/code-architect.md#L5-L5
  • agents/code-explorer.md#L5-L5
  • agents/code-reviewer.md#L4-L4
  • agents/code-simplifier.md#L5-L5
  • agents/opensource-forker.md#L4-L4
  • agents/opensource-packager.md#L4-L4
  • agents/opensource-sanitizer.md#L4-L4
  • agents/performance-optimizer.md#L4-L4
  • agents/php-reviewer.md#L4-L4
  • agents/planner.md#L4-L4
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/chief-of-staff.md` at line 4, Convert the tools frontmatter value from
a scalar to a YAML array in agents/chief-of-staff.md:4-4,
agents/code-architect.md:5-5, agents/code-explorer.md:5-5,
agents/code-reviewer.md:4-4, agents/code-simplifier.md:5-5,
agents/opensource-forker.md:4-4, agents/opensource-packager.md:4-4,
agents/opensource-sanitizer.md:4-4, agents/performance-optimizer.md:4-4,
agents/php-reviewer.md:4-4, and agents/planner.md:4-4, preserving each agent’s
existing tool entries.

Source: Path instructions

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/code-architect.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: code-architect
description: Designs feature architectures by analyzing existing codebase patterns and conventions, then providing implementation blueprints with concrete files, interfaces, data flow, and build order.
model: sonnet
tools: [Read, Grep, Glob, Bash]
tools: Read, Grep, Glob, Bash
---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/code-explorer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: code-explorer
description: Deeply analyzes existing codebase features by tracing execution paths, mapping architecture layers, and documenting dependencies to inform new development.
model: sonnet
tools: [Read, Grep, Glob]
tools: Read, Grep, Glob
---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/code-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: code-reviewer
description: Expert code review specialist. Proactively reviews code for quality, security, and maintainability. Use immediately after writing or modifying code. MUST BE USED for all code changes.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/code-simplifier.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: code-simplifier
description: Simplifies and refines code for clarity, consistency, and maintainability while preserving behavior. Focus on recently modified code unless instructed otherwise.
model: sonnet
tools: [Read, Write, Edit, Bash, Grep, Glob]
tools: Read, Write, Edit, Bash, Grep, Glob
---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/comment-analyzer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: comment-analyzer
description: Analyze code comments for accuracy, completeness, maintainability, and comment rot risk.
model: haiku
tools: [Read, Grep, Glob]
tools: Read, Grep, Glob

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Update downstream parsers before switching agent metadata to a scalar tools field. scripts/gemini-adapt-agents.js and scripts/lib/agent-compress.js need to accept both formats, or this change breaks agent discovery/tool availability across the affected files.

📍 Affects 11 files
  • agents/comment-analyzer.md#L5-L5 (this comment)
  • agents/conversation-analyzer.md#L5-L5
  • agents/cpp-build-resolver.md#L4-L4
  • agents/cpp-reviewer.md#L4-L4
  • agents/csharp-reviewer.md#L4-L4
  • agents/pr-test-analyzer.md#L5-L5
  • agents/python-reviewer.md#L4-L4
  • agents/pytorch-build-resolver.md#L4-L4
  • agents/react-build-resolver.md#L4-L4
  • agents/react-reviewer.md#L4-L4
  • agents/refactor-cleaner.md#L4-L4
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/comment-analyzer.md` at line 5, Update scripts/gemini-adapt-agents.js
and scripts/lib/agent-compress.js to accept both scalar and array forms of the
tools metadata before relying on the new scalar format. Apply this compatibility
change for agents/comment-analyzer.md:5, agents/conversation-analyzer.md:5,
agents/cpp-build-resolver.md:4, agents/cpp-reviewer.md:4,
agents/csharp-reviewer.md:4, agents/pr-test-analyzer.md:5,
agents/python-reviewer.md:4, agents/pytorch-build-resolver.md:4,
agents/react-build-resolver.md:4, agents/react-reviewer.md:4, and
agents/refactor-cleaner.md:4; no direct changes are required in those agent
files.

---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/conversation-analyzer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
name: conversation-analyzer
description: Use this agent when analyzing conversation transcripts to find behaviors worth preventing with hooks. Triggered by /hookify without arguments.
model: haiku
tools: [Read, Grep]
tools: Read, Grep
---

## Prompt Defense Baseline
Expand Down
2 changes: 1 addition & 1 deletion agents/cpp-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: cpp-build-resolver
description: C++ build, CMake, and compilation error resolution specialist. Fixes build errors, linker issues, and template errors with minimal changes. Use when C++ builds fail.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/cpp-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: cpp-reviewer
description: Expert C++ code reviewer specializing in memory safety, modern C++ idioms, concurrency, and performance. Use for all C++ code changes. MUST BE USED for C++ projects.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/csharp-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: csharp-reviewer
description: Expert C# code reviewer specializing in .NET conventions, async patterns, security, nullable reference types, and performance. Use for all C# code changes. MUST BE USED for C# projects.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/dart-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: dart-build-resolver
description: Dart/Flutter build, analysis, and dependency error resolution specialist. Fixes `dart analyze` errors, Flutter compilation failures, pub dependency conflicts, and build_runner issues with minimal, surgical changes. Use when Dart/Flutter builds fail.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Update the dashboard parser to support the new scalar contract.

All these agents now use comma-separated scalar tools values, but scripts/dashboard-web.js:24-51 only recognizes arrays and silently exposes t: []. This creates a cross-layer metadata mismatch: agent definitions retain tools, while the dashboard reports none.

  • agents/dart-build-resolver.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/database-reviewer.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/django-build-resolver.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/django-reviewer.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/doc-updater.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/rust-build-resolver.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/rust-reviewer.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/security-reviewer.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/seo-specialist.md#L4-L4: parse the scalar tool list in the dashboard loader.
  • agents/silent-failure-hunter.md#L5-L5: parse the scalar tool list in the dashboard loader.
  • agents/spec-miner.md#L5-L5: parse the scalar tool list in the dashboard loader while retaining its existing tool guardrails.
📍 Affects 11 files
  • agents/dart-build-resolver.md#L4-L4 (this comment)
  • agents/database-reviewer.md#L4-L4
  • agents/django-build-resolver.md#L4-L4
  • agents/django-reviewer.md#L4-L4
  • agents/doc-updater.md#L4-L4
  • agents/rust-build-resolver.md#L4-L4
  • agents/rust-reviewer.md#L4-L4
  • agents/security-reviewer.md#L4-L4
  • agents/seo-specialist.md#L4-L4
  • agents/silent-failure-hunter.md#L5-L5
  • agents/spec-miner.md#L5-L5
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/dart-build-resolver.md` at line 4, The dashboard loader/parser in
scripts/dashboard-web.js must parse comma-separated scalar tools values instead
of treating them as empty arrays, while preserving existing array handling and
spec-miner’s tool guardrails. Apply this shared loader fix for
agents/dart-build-resolver.md:4, agents/database-reviewer.md:4,
agents/django-build-resolver.md:4, agents/django-reviewer.md:4,
agents/doc-updater.md:4, agents/rust-build-resolver.md:4,
agents/rust-reviewer.md:4, agents/security-reviewer.md:4,
agents/seo-specialist.md:4, agents/silent-failure-hunter.md:5, and
agents/spec-miner.md:5; these sites require no direct edits.

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/database-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: database-reviewer
description: PostgreSQL database specialist for query optimization, schema design, security, and performance. Use PROACTIVELY when writing SQL, creating migrations, designing schemas, or troubleshooting database performance. Incorporates Supabase best practices.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/django-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: django-build-resolver
description: Django/Python build, migration, and dependency error resolution specialist. Fixes pip/Poetry errors, migration conflicts, import errors, Django configuration issues, and collectstatic failures with minimal changes. Use when Django setup or startup fails.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/django-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: django-reviewer
description: Expert Django code reviewer specializing in ORM correctness, DRF patterns, migration safety, security misconfigurations, and production-grade Django practices. Use for all Django code changes. MUST BE USED for Django projects.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/doc-updater.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: doc-updater
description: Documentation and codemap specialist. Use PROACTIVELY for updating codemaps and documentation. Runs /update-codemaps and /update-docs, generates docs/CODEMAPS/*, updates READMEs and guides.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: haiku
---

Expand Down
2 changes: 1 addition & 1 deletion agents/docs-lookup.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Update repository consumers before converting all agent metadata.

The new values are YAML strings, but scripts/dashboard-web.js and scripts/lib/agent-compress.js only retain array-valued tools. These agents will therefore expose empty tool lists internally, contradicting tests/scripts/dashboard-web.test.js:110-126.

  • agents/docs-lookup.md#L4-L4: Add scalar-to-array normalization before publishing this agent's tools.
  • agents/e2e-runner.md#L4-L4: Preserve Read, Write, Edit, Bash, Grep, and Glob in internal metadata.
  • agents/fastapi-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
  • agents/flutter-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
  • agents/fsharp-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
  • agents/swift-build-resolver.md#L4-L4: Preserve all six declared tools.
  • agents/swift-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
  • agents/tdd-guide.md#L4-L4: Preserve Read, Write, Edit, Bash, and Grep.
  • agents/type-design-analyzer.md#L5-L5: Preserve Read, Grep, and Glob.
  • agents/typescript-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
  • agents/vue-reviewer.md#L4-L4: Preserve Read, Grep, Glob, and Bash.
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/docs-lookup.md` at line 1, Update the dashboard and agent-compression
consumers to normalize scalar YAML tool values into single-element arrays before
publishing or storing agent metadata. Preserve existing array-valued handling,
and ensure the listed agents retain every declared tool so their internal
metadata matches the expected tool lists in the dashboard tests.

name: docs-lookup
description: When the user asks how to use a library, framework, or API or needs up-to-date code examples, use Context7 MCP to fetch current documentation and return answers with examples. Invoke for docs/API/setup questions.
tools: ["Read", "Grep", "mcp__context7__resolve-library-id", "mcp__context7__query-docs"]
tools: Read, Grep, mcp__context7__resolve-library-id, mcp__context7__query-docs

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Preserve tool metadata for repository consumers.

This scalar value is valid YAML, but scripts/dashboard-web.js and scripts/lib/agent-compress.js currently keep tools only when it is an array. After this change, this agent will be published with an empty tool list, while tests/scripts/dashboard-web.test.js:110-126 still requires arrays. Update those consumers to split comma-separated strings, or defer this conversion until they support both formats.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/docs-lookup.md` at line 4, Update the tools metadata consumers in
scripts/dashboard-web.js and scripts/lib/agent-compress.js to accept scalar
comma-separated values as well as arrays, splitting and normalizing the string
into the expected tool list while preserving existing array handling. Ensure the
published agent retains its tools metadata and satisfies the array expectations
in the dashboard tests.

model: haiku
---

Expand Down
2 changes: 1 addition & 1 deletion agents/e2e-runner.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: e2e-runner
description: End-to-end testing specialist using Vercel Agent Browser (preferred) with Playwright fallback. Use PROACTIVELY for generating, maintaining, and running E2E tests. Manages test journeys, quarantines flaky tests, uploads artifacts (screenshots, videos, traces), and ensures critical user flows work.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Keep internal loaders compatible with the new scalar format.

scripts/dashboard-web.js and scripts/lib/agent-compress.js discard non-array tools values, so this edit will make e2e-runner appear to have no tools internally. Add scalar parsing support before converting the agent files.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/e2e-runner.md` at line 4, Update the internal agent-file conversion
flow used by scripts/dashboard-web.js and scripts/lib/agent-compress.js to
preserve scalar tools values such as the e2e-runner configuration. Add scalar
parsing support before converting agent files, while retaining existing array
handling.

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/fastapi-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: fastapi-reviewer
description: Reviews FastAPI applications for async correctness, dependency injection, Pydantic schemas, security, OpenAPI quality, testing, and production readiness.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Update consumers before changing the frontmatter type.

The repository's loadAgents and loadAgent paths convert non-array tools values to []. Consequently, this agent will lose Read, Grep, Glob, and Bash in internal metadata despite the frontmatter remaining syntactically valid.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/fastapi-reviewer.md` at line 4, Update the consumers in loadAgents and
loadAgent to preserve non-array tools values as valid tool metadata, rather than
converting them to []. Ensure Read, Grep, Glob, and Bash remain available after
loading this agent before changing the frontmatter type.

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/flutter-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: flutter-reviewer
description: Flutter and Dart code reviewer. Reviews Flutter code for widget best practices, state management patterns, Dart idioms, performance pitfalls, accessibility, and clean architecture violations. Library-agnostic — works with any state management solution and tooling.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Do not ship the scalar format without parser support.

The internal frontmatter consumers shown in context treat this value as a string and then coerce it to an empty array. That silently removes this agent's tool permissions from repository-generated metadata.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/flutter-reviewer.md` at line 4, The tool configuration must use a
parser-supported collection format rather than a scalar string so frontmatter
consumers preserve the agent’s tool permissions. Update the tools declaration in
agents/flutter-reviewer.md to match the array format expected by the existing
frontmatter parsing logic, while retaining the listed Read, Grep, Glob, and Bash
tools.

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/fsharp-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: fsharp-reviewer
description: Expert F# code reviewer specializing in functional idioms, type safety, pattern matching, computation expressions, and performance. Use for all F# code changes. MUST BE USED for F# projects.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | 🏗️ Heavy lift

Resolve the array-versus-string contract before merging.

tests/scripts/dashboard-web.test.js documents array-valued tools, and both supplied loaders discard strings. Convert the loaders to normalize either representation before applying this change, with a regression test for comma-separated values.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/fsharp-reviewer.md` at line 4, Update both supplied tool loaders to
normalize configured tools from either arrays or comma-separated strings before
applying the new behavior, preserving array-valued inputs. Add a regression test
in tests/scripts/dashboard-web.test.js covering comma-separated tool values and
verifying they are parsed into the expected array.

model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/gan-evaluator.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: gan-evaluator
description: "GAN Harness — Evaluator agent. Tests the live running application via Playwright, scores against rubric, and provides actionable feedback to the Generator."
tools: ["Read", "Write", "Bash", "Grep", "Glob"]
tools: Read, Write, Bash, Grep, Glob
model: sonnet
color: red
---
Expand Down
2 changes: 1 addition & 1 deletion agents/gan-generator.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: gan-generator
description: "GAN Harness — Generator agent. Implements features according to the spec, reads evaluator feedback, and iterates until quality threshold is met."
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob

@coderabbitai coderabbitai Bot Jul 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

rg -n -C3 'agent\.tools|parseFrontmatter|agent-compress' scripts agents

Repository: affaan-m/ECC

Length of output: 2659

Split scalar tools values before loading agents. scripts/lib/agent-compress.js passes frontmatter.tools through unchanged, so tools: Read, Write, Edit, Bash, Grep, Glob will stay a string instead of the array shape the rest of the agent metadata expects. Normalize comma-separated values in the loader, or keep this field as YAML list syntax.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agents/gan-generator.md` at line 4, Normalize comma-separated scalar values
from frontmatter.tools into an array before agents are loaded, within the loader
used by scripts/lib/agent-compress.js, trimming each tool name and preserving
existing array values; alternatively, change the metadata to YAML list syntax so
the loader receives the expected array shape.

model: sonnet
color: green
---
Expand Down
2 changes: 1 addition & 1 deletion agents/gan-planner.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: gan-planner
description: "GAN Harness — Planner agent. Expands a one-line prompt into a full product specification with features, sprints, evaluation criteria, and design direction."
tools: ["Read", "Write", "Grep", "Glob"]
tools: Read, Write, Grep, Glob
model: sonnet
color: purple
---
Expand Down
2 changes: 1 addition & 1 deletion agents/go-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: go-build-resolver
description: Go build, vet, and compilation error resolution specialist. Fixes build errors, go vet issues, and linter warnings with minimal changes. Use when Go builds fail.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/go-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: go-reviewer
description: Expert Go code reviewer specializing in idiomatic Go, concurrency patterns, error handling, and performance. Use for all Go code changes. MUST BE USED for Go projects.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/harmonyos-app-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: harmonyos-app-resolver
description: HarmonyOS application development expert specializing in ArkTS and ArkUI. Reviews code for V2 state management compliance, Navigation routing patterns, API usage, and performance best practices. Use for HarmonyOS/OpenHarmony projects.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/harness-optimizer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: harness-optimizer
description: Analyze and improve the local agent harness configuration for reliability, cost, and throughput.
tools: ["Read", "Grep", "Glob", "Bash", "Edit"]
tools: Read, Grep, Glob, Bash, Edit
model: sonnet
color: teal
---
Expand Down
2 changes: 1 addition & 1 deletion agents/healthcare-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: healthcare-reviewer
description: Reviews healthcare application code for clinical safety, CDSS accuracy, PHI compliance, and medical data integrity. Specialized for EMR/EHR, clinical decision support, and health information systems.
tools: ["Read", "Grep", "Glob"]
tools: Read, Grep, Glob
model: opus
---

Expand Down
2 changes: 1 addition & 1 deletion agents/homelab-architect.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: homelab-architect
description: Designs home and small-lab network plans from hardware inventory, goals, and operator experience level, with safe staged changes and rollback guidance.
tools: ["Read", "Grep"]
tools: Read, Grep
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/java-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: java-build-resolver
description: Java/Maven/Gradle build, compilation, and dependency error resolution specialist. Automatically detects Spring Boot or Quarkus and applies framework-specific fixes. Fixes build errors, Java compiler errors, and Maven/Gradle issues with minimal changes. Use when Java builds fail.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/java-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: java-reviewer
description: Expert Java code reviewer for Spring Boot and Quarkus projects. Automatically detects the framework and applies the appropriate review rules. Covers layered architecture, JPA/Panache, MongoDB, security, and concurrency. MUST BE USED for all Java code changes.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/kotlin-build-resolver.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: kotlin-build-resolver
description: Kotlin/Gradle build, compilation, and dependency error resolution specialist. Fixes build errors, Kotlin compiler errors, and Gradle issues with minimal changes. Use when Kotlin builds fail.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/kotlin-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: kotlin-reviewer
description: Kotlin and Android/KMP code reviewer. Reviews Kotlin code for idiomatic patterns, coroutine safety, Compose best practices, clean architecture violations, and common Android pitfalls.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/loop-operator.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: loop-operator
description: Operate autonomous agent loops, monitor progress, and intervene safely when loops stall.
tools: ["Read", "Grep", "Glob", "Bash", "Edit"]
tools: Read, Grep, Glob, Bash, Edit
model: sonnet
color: orange
---
Expand Down
2 changes: 1 addition & 1 deletion agents/marketing-agent.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: marketing-agent
description: Marketing strategist and copywriter for campaign planning, audience research, positioning, copy creation, and content review. Covers landing pages, email sequences, social posts, ad copy, short-form video scripts, and content calendars. Use when the user wants to plan or execute a product launch or marketing campaign.
tools: ["Read", "Grep", "Glob", "WebSearch", "WebFetch"]
tools: Read, Grep, Glob, WebSearch, WebFetch
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/mle-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: mle-reviewer
description: Production machine-learning engineering reviewer for data contracts, feature pipelines, training reproducibility, offline/online evaluation, model serving, monitoring, and rollback. Use when ML, MLOps, model training, inference, feature store, or evaluation code changes.
tools: ["Read", "Grep", "Glob", "Bash"]
tools: Read, Grep, Glob, Bash
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/network-architect.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: network-architect
description: Designs enterprise or multi-site network architecture from requirements, using existing network skills for focused routing, validation, automation, and troubleshooting detail.
tools: ["Read", "Grep"]
tools: Read, Grep
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/network-config-reviewer.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: network-config-reviewer
description: Reviews router and switch configurations for security, correctness, stale references, risky change-window commands, and missing operational guardrails.
tools: ["Read", "Grep"]
tools: Read, Grep
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/network-troubleshooter.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: network-troubleshooter
description: Diagnoses network connectivity, routing, DNS, interface, and policy symptoms with a read-only OSI-layer workflow and evidence-backed root cause summary.
tools: ["Read", "Bash", "Grep"]
tools: Read, Bash, Grep
model: sonnet
---

Expand Down
2 changes: 1 addition & 1 deletion agents/opensource-forker.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
name: opensource-forker
description: Fork any project for open-sourcing. Copies files, strips secrets and credentials (20+ patterns), replaces internal references with placeholders, generates .env.example, and cleans git history. First stage of the opensource-pipeline skill.
tools: ["Read", "Write", "Edit", "Bash", "Grep", "Glob"]
tools: Read, Write, Edit, Bash, Grep, Glob
model: haiku
---

Expand Down
Loading