fix: address CodeRabbit review — closeDb try/finally, stale docstring, SKILL.md kind

- closeDb: wrap PRAGMA optimize in try/finally so db.close() always runs
- run-index.ts: update RunIndexOptions.files docstring (no longer filtered)
- SKILL.md: remove stale `variable` from symbols.kind (agents + templates)

Author: SutuSebastian

.agents/skills/codemap/SKILL.md

@@ -77,17 +77,17 @@ LIMIT 10
 
 ### `symbols` — Functions, types, interfaces, enums, constants, classes
 
-| Column            | Type       | Description                                                           |
-| ----------------- | ---------- | --------------------------------------------------------------------- |
-| id                | INTEGER PK | Auto-increment ID                                                     |
-| file_path         | TEXT FK    | References `files(path)`                                              |
-| name              | TEXT       | Symbol name                                                           |
-| kind              | TEXT       | `function`, `class`, `type`, `interface`, `enum`, `const`, `variable` |
-| line_start        | INTEGER    | Start line (1-based)                                                  |
-| line_end          | INTEGER    | End line (1-based)                                                    |
-| signature         | TEXT       | e.g. `createHandler()`, `type UserProps`                              |
-| is_exported       | INTEGER    | 1 if exported                                                         |
-| is_default_export | INTEGER    | 1 if default export                                                   |
+| Column            | Type       | Description                                               |
+| ----------------- | ---------- | --------------------------------------------------------- |
+| id                | INTEGER PK | Auto-increment ID                                         |
+| file_path         | TEXT FK    | References `files(path)`                                  |
+| name              | TEXT       | Symbol name                                               |
+| kind              | TEXT       | `function`, `class`, `type`, `interface`, `enum`, `const` |
+| line_start        | INTEGER    | Start line (1-based)                                      |
+| line_end          | INTEGER    | End line (1-based)                                        |
+| signature         | TEXT       | e.g. `createHandler()`, `type UserProps`                  |
+| is_exported       | INTEGER    | 1 if exported                                             |
+| is_default_export | INTEGER    | 1 if default export                                       |
 
 ### `imports` — Import statements
 

src/application/run-index.ts

@@ -40,7 +40,7 @@ export interface RunIndexOptions {
   mode?: IndexMode;
   /**
    * Paths relative to the project root; used only when `mode === "files"`.
-   * Non-indexable extensions are filtered out.
+   * All paths are forwarded as-is; non-standard extensions are indexed as text.
    */
   files?: string[];
   /**

src/db.ts

@@ -18,11 +18,14 @@ export function openDb(): CodemapDatabase {
 }
 
 export function closeDb(db: CodemapDatabase, opts?: { readonly?: boolean }) {
-  if (!opts?.readonly) {
-    db.run("PRAGMA analysis_limit = 400");
-    db.run("PRAGMA optimize");
+  try {
+    if (!opts?.readonly) {
+      db.run("PRAGMA analysis_limit = 400");
+      db.run("PRAGMA optimize");
+    }
+  } finally {
+    db.close();
   }
-  db.close();
 }
 
 export function createTables(db: CodemapDatabase) {

templates/agents/skills/codemap/SKILL.md

@@ -77,17 +77,17 @@ LIMIT 10
 
 ### `symbols` — Functions, types, interfaces, enums, constants, classes
 
-| Column            | Type       | Description                                                           |
-| ----------------- | ---------- | --------------------------------------------------------------------- |
-| id                | INTEGER PK | Auto-increment ID                                                     |
-| file_path         | TEXT FK    | References `files(path)`                                              |
-| name              | TEXT       | Symbol name                                                           |
-| kind              | TEXT       | `function`, `class`, `type`, `interface`, `enum`, `const`, `variable` |
-| line_start        | INTEGER    | Start line (1-based)                                                  |
-| line_end          | INTEGER    | End line (1-based)                                                    |
-| signature         | TEXT       | e.g. `createHandler()`, `type UserProps`                              |
-| is_exported       | INTEGER    | 1 if exported                                                         |
-| is_default_export | INTEGER    | 1 if default export                                                   |
+| Column            | Type       | Description                                               |
+| ----------------- | ---------- | --------------------------------------------------------- |
+| id                | INTEGER PK | Auto-increment ID                                         |
+| file_path         | TEXT FK    | References `files(path)`                                  |
+| name              | TEXT       | Symbol name                                               |
+| kind              | TEXT       | `function`, `class`, `type`, `interface`, `enum`, `const` |
+| line_start        | INTEGER    | Start line (1-based)                                      |
+| line_end          | INTEGER    | End line (1-based)                                        |
+| signature         | TEXT       | e.g. `createHandler()`, `type UserProps`                  |
+| is_exported       | INTEGER    | 1 if exported                                             |
+| is_default_export | INTEGER    | 1 if default export                                       |
 
 ### `imports` — Import statements