From 99d443b16e1fb7833378e7763e256f977f05d18e Mon Sep 17 00:00:00 2001
From: Affaan Mustafa <affaan@dcube.ai>
Date: Thu, 12 Mar 2026 23:53:23 -0700
Subject: [PATCH] fix: align kotlin diagnostics and heading hierarchy

---
 agents/kotlin-build-resolver.md |  2 +-
 commands/kotlin-build.md        |  2 +-
 skills/kotlin-testing/SKILL.md  | 46 ++++++++++++++++-----------------
 3 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/agents/kotlin-build-resolver.md b/agents/kotlin-build-resolver.md
index 705afd32..e8c45599 100644
--- a/agents/kotlin-build-resolver.md
+++ b/agents/kotlin-build-resolver.md
@@ -25,7 +25,7 @@ Run these in order:
 ./gradlew build 2>&1
 ./gradlew detekt 2>&1 || echo "detekt not configured"
 ./gradlew ktlintCheck 2>&1 || echo "ktlint not configured"
-./gradlew dependencies --configuration runtimeClasspath 2>/dev/null | head -100
+./gradlew dependencies --configuration runtimeClasspath 2>&1 | head -100
 ```
 
 ## Resolution Workflow
diff --git a/commands/kotlin-build.md b/commands/kotlin-build.md
index 01c75ccb..70709c8d 100644
--- a/commands/kotlin-build.md
+++ b/commands/kotlin-build.md
@@ -34,7 +34,7 @@ Use `/kotlin-build` when:
 ./gradlew ktlintCheck 2>&1 || echo "ktlint not configured"
 
 # Dependency issues
-./gradlew dependencies --configuration runtimeClasspath 2>/dev/null | head -100
+./gradlew dependencies --configuration runtimeClasspath 2>&1 | head -100
 
 # Optional deep refresh when caches or dependency metadata are suspect
 ./gradlew build --refresh-dependencies
diff --git a/skills/kotlin-testing/SKILL.md b/skills/kotlin-testing/SKILL.md
index ae326216..8819c4f4 100644
--- a/skills/kotlin-testing/SKILL.md
+++ b/skills/kotlin-testing/SKILL.md
@@ -40,7 +40,7 @@ The following sections contain detailed, runnable examples for each testing patt
 
 ### TDD Workflow for Kotlin
 
-### The RED-GREEN-REFACTOR Cycle
+#### The RED-GREEN-REFACTOR Cycle
 
 ```
 RED     -> Write a failing test first
@@ -49,7 +49,7 @@ REFACTOR -> Improve code while keeping tests green
 REPEAT  -> Continue with next requirement
 ```
 
-### Step-by-Step TDD in Kotlin
+#### Step-by-Step TDD in Kotlin
 
 ```kotlin
 // Step 1: Define the interface/signature
@@ -107,7 +107,7 @@ fun validateEmail(email: String): Result<String> {
 
 ### Kotest Spec Styles
 
-### StringSpec (Simplest)
+#### StringSpec (Simplest)
 
 ```kotlin
 class CalculatorTest : StringSpec({
@@ -125,7 +125,7 @@ class CalculatorTest : StringSpec({
 })
 ```
 
-### FunSpec (JUnit-like)
+#### FunSpec (JUnit-like)
 
 ```kotlin
 class UserServiceTest : FunSpec({
@@ -151,7 +151,7 @@ class UserServiceTest : FunSpec({
 })
 ```
 
-### BehaviorSpec (BDD Style)
+#### BehaviorSpec (BDD Style)
 
 ```kotlin
 class OrderServiceTest : BehaviorSpec({
@@ -193,7 +193,7 @@ class OrderServiceTest : BehaviorSpec({
 })
 ```
 
-### DescribeSpec (RSpec Style)
+#### DescribeSpec (RSpec Style)
 
 ```kotlin
 class UserValidatorTest : DescribeSpec({
@@ -224,7 +224,7 @@ class UserValidatorTest : DescribeSpec({
 
 ### Kotest Matchers
 
-### Core Matchers
+#### Core Matchers
 
 ```kotlin
 import io.kotest.matchers.shouldBe
@@ -272,7 +272,7 @@ shouldNotThrow<Exception> {
 }
 ```
 
-### Custom Matchers
+#### Custom Matchers
 
 ```kotlin
 fun beActiveUser() = object : Matcher<User> {
@@ -289,7 +289,7 @@ user should beActiveUser()
 
 ### MockK
 
-### Basic Mocking
+#### Basic Mocking
 
 ```kotlin
 class UserServiceTest : FunSpec({
@@ -321,7 +321,7 @@ class UserServiceTest : FunSpec({
 })
 ```
 
-### Coroutine Mocking
+#### Coroutine Mocking
 
 ```kotlin
 class AsyncUserServiceTest : FunSpec({
@@ -349,7 +349,7 @@ class AsyncUserServiceTest : FunSpec({
 })
 ```
 
-### Argument Capture
+#### Argument Capture
 
 ```kotlin
 test("save captures the user argument") {
@@ -364,7 +364,7 @@ test("save captures the user argument") {
 }
 ```
 
-### Spy and Partial Mocking
+#### Spy and Partial Mocking
 
 ```kotlin
 test("spy on real object") {
@@ -382,7 +382,7 @@ test("spy on real object") {
 
 ### Coroutine Testing
 
-### runTest for Suspend Functions
+#### runTest for Suspend Functions
 
 ```kotlin
 import kotlinx.coroutines.test.runTest
@@ -413,7 +413,7 @@ class CoroutineServiceTest : FunSpec({
 })
 ```
 
-### Testing Flows
+#### Testing Flows
 
 ```kotlin
 import io.kotest.matchers.collections.shouldContainInOrder
@@ -459,7 +459,7 @@ class FlowServiceTest : FunSpec({
 })
 ```
 
-### TestDispatcher
+#### TestDispatcher
 
 ```kotlin
 import kotlinx.coroutines.test.StandardTestDispatcher
@@ -487,7 +487,7 @@ class DispatcherTest : FunSpec({
 
 ### Property-Based Testing
 
-### Kotest Property Testing
+#### Kotest Property Testing
 
 ```kotlin
 import io.kotest.core.spec.style.FunSpec
@@ -527,7 +527,7 @@ class PropertyTest : FunSpec({
 })
 ```
 
-### Custom Generators
+#### Custom Generators
 
 ```kotlin
 val userArb: Arb<User> = Arb.bind(
@@ -553,7 +553,7 @@ val moneyArb: Arb<Money> = Arb.bind(
 
 ### Data-Driven Testing
 
-### withData in Kotest
+#### withData in Kotest
 
 ```kotlin
 class ParserTest : FunSpec({
@@ -585,7 +585,7 @@ class ParserTest : FunSpec({
 
 ### Test Lifecycle and Fixtures
 
-### BeforeTest / AfterTest
+#### BeforeTest / AfterTest
 
 ```kotlin
 class DatabaseTest : FunSpec({
@@ -627,7 +627,7 @@ class DatabaseTest : FunSpec({
 })
 ```
 
-### Kotest Extensions
+#### Kotest Extensions
 
 ```kotlin
 // Reusable test extension
@@ -656,7 +656,7 @@ class UserRepositoryTest : FunSpec({
 
 ### Kover Coverage
 
-### Gradle Configuration
+#### Gradle Configuration
 
 ```kotlin
 // build.gradle.kts
@@ -684,7 +684,7 @@ kover {
 }
 ```
 
-### Coverage Commands
+#### Coverage Commands
 
 ```bash
 # Run tests with coverage
@@ -702,7 +702,7 @@ kover {
 # Windows: start build/reports/kover/html/index.html
 ```
 
-### Coverage Targets
+#### Coverage Targets
 
 | Code Type | Target |
 |-----------|--------|