quietlight/skraak_mcp - Change 42UVPZU3N47BJIV5KBUN4KDB2JKP5QCGMNEOTZTLWP2XNUCNOARAC

deleted the mcp tool import_file (1 file), beware as model did a run around

Created by quietlight on February 20, 2026

42UVPZU3N47BJIV5KBUN4KDB2JKP5QCGMNEOTZTLWP2XNUCNOARAC

Dependencies

In channels

main

Change contents

Replacement in tools/import_file.go at line 56 [3.1]

B:BD[3.1612] → [4.18696:18790]

// ImportFile imports a single WAV audio file into the database with full metadata extraction

[3.1612]

[5.4274]

// ImportFile imports a single WAV file into the database with full metadata extraction

Deletion in shell_scripts/test_import.sh at line 37 [6.16388]

B:BD[6.17848] → [6.17848:17884]

B:BD[6.17884] → [7.41:79]

∅:D[7.79] → [6.17916:18000]

B:BD[6.17916] → [6.17916:18000]

B:BD[6.18000] → [7.80:311]

∅:D[7.311] → [6.18225:18353]

B:BD[6.18225] → [6.18225:18353]

B:BD[6.18353] → [7.312:525]

∅:D[7.525] → [6.18560:18639]

B:BD[6.18560] → [6.18560:18639]

echo ""
# === IMPORT_FILE TOOL ===
echo "=== import_audio_file Tool ==="
echo ""
# Test 1: Non-existent file
echo "Test 1: Non-existent file (should fail)"
result=$(send_request "tools/call" '{"name":"import_audio_file","arguments":{"file_path":"/nonexistent/path/to/file.wav","dataset_id":"'"$DATASET_ID"'","location_id":"'"$LOCATION_ID"'","cluster_id":"'"$CLUSTER_ID"'"}}' "$DB_PATH")
run_test "Reject non-existent file" "false" "$result"
# Test 2: Non-WAV file
echo ""
echo "Test 2: Non-WAV file (should fail)"
result=$(send_request "tools/call" '{"name":"import_audio_file","arguments":{"file_path":"/etc/passwd","dataset_id":"'"$DATASET_ID"'","location_id":"'"$LOCATION_ID"'","cluster_id":"'"$CLUSTER_ID"'"}}' "$DB_PATH")
run_test "Reject non-WAV file" "false" "$result"
# Test 3: Invalid dataset ID

Deletion in shell_scripts/test_import.sh at line 38 [6.16388]

B:BD[6.18647] → [6.18647:18695]

B:BD[6.18695] → [7.526:739]

∅:D[7.739] → [6.18902:19043]

B:BD[6.18902] → [6.18902:19043]

B:BD[6.19043] → [7.740:953]

∅:D[7.953] → [6.19250:19305]

B:BD[6.19250] → [6.19250:19305]

echo "Test 3: Invalid dataset_id (should fail)"
result=$(send_request "tools/call" '{"name":"import_audio_file","arguments":{"file_path":"/tmp/test.wav","dataset_id":"INVALID123456","location_id":"'"$LOCATION_ID"'","cluster_id":"'"$CLUSTER_ID"'"}}' "$DB_PATH")
run_test "Reject invalid dataset_id" "false" "$result"
# Test 4: Invalid cluster ID
echo ""
echo "Test 4: Invalid cluster_id (should fail)"
result=$(send_request "tools/call" '{"name":"import_audio_file","arguments":{"file_path":"/tmp/test.wav","dataset_id":"'"$DATASET_ID"'","location_id":"'"$LOCATION_ID"'","cluster_id":"INVALID123456"}}' "$DB_PATH")
run_test "Reject invalid cluster_id" "false" "$result"

Deletion in shell_scripts/test_import.sh at line 40 [6.16388]
B:BD[6.21507] → [6.21507:21515]
```
echo ""
```

Replacement in shell_scripts/test_import.sh at line 43 [6.16388]

B:BD[6.21563] → [8.41:120]

# Test 5: Non-existent folder
echo "Test 5: Non-existent folder (should fail)"

[6.21563]

[6.21642]

# Test 1: Non-existent folder
echo "Test 1: Non-existent folder (should fail)"

Replacement in shell_scripts/test_import.sh at line 48 [6.16388]
B:BD[6.21923] → [8.121:151]
```
# Test 6: Invalid location ID
```
[6.21923]
[6.21953]
```
# Test 2: Invalid location ID
```
Replacement in shell_scripts/test_import.sh at line 50 [6.16388]
B:BD[6.21961] → [8.152:201]
```
echo "Test 6: Invalid location_id (should fail)"
```
[6.21961]
[6.22010]
```
echo "Test 2: Invalid location_id (should fail)"
```

Replacement in shell_scripts/TESTING.md at line 8 [10.57101]

B:BD[9.46552] → [8.360:450]

- **Import tools (3)**: `import_audio_files`, `import_audio_file`, `import_ml_selections`

[9.46552]

[11.32185]

- **Import tools (2 MCP)**: `import_audio_files`, `import_ml_selections`

Deletion in cmd/mcp.go at line 94 [4.1647]

B:BD[4.4211] → [4.4211:4244]

B:BD[4.4244] → [7.1088:1124]

∅:D[7.1124] → [4.4274:4490]

B:BD[4.4274] → [4.4274:4490]

B:BD[4.4490] → [7.1125:1149]


	mcp.AddTool(server, &mcp.Tool{
		Name:        "import_audio_file",
		Description: "Import a single WAV file into the database. Automatically parses AudioMoth and filename timestamps, calculates hash, extracts metadata, and computes astronomical data. Skips if duplicate (by hash).",
	}, mcpImportAudioFile)

Deletion in cmd/mcp.go at line 156 [4.1647]

B:BD[4.8176] → [7.1150:1305]

∅:D[7.1305] → [4.8326:8417]

B:BD[4.8326] → [4.8326:8417]

func mcpImportAudioFile(ctx context.Context, req *mcp.CallToolRequest, input tools.ImportFileInput) (*mcp.CallToolResult, tools.ImportFileOutput, error) {
	output, err := tools.ImportFile(ctx, input)
	return &mcp.CallToolResult{}, output, err
}

Deletion in README.md at line 96 [13.334405]
B:BD[12.152604] → [7.1308:1357]
```
- `import_audio_file` - Import a single WAV file
```
Replacement in CLAUDE.md at line 119 [13.363912]
B:BD[14.1677] → [15.4739:4760]
```
### Import Tools (5)
```
[14.1677]
[16.11589]
```
### Import Tools (4 MCP tools, 1 CLI-only)
```

Replacement in CLAUDE.md at line 121 [13.363912]

B:BD[16.11620] → [15.4761:4863]

- Structured imports: `import_audio_files`, `import_file`, `bulk_file_import`, `import_ml_selections`

[16.11620]

[15.4863]

- Structured imports: `import_audio_files`, `bulk_file_import`, `import_ml_selections`

Replacement in CLAUDE.md at line 124 [13.363912]
B:BD[14.1864] → [15.4910:4921]
```
**Tools:**
```
[14.1864]
[15.4921]
```
**MCP Tools:**
```
Deletion in CLAUDE.md at line 126 [13.363912]
B:BD[15.4978] → [15.4978:5021]
```
- `import_audio_file` - Single file import
```

Replacement in CLAUDE.md at line 128 [13.363912]

B:BD[15.5174] → [15.5174:5240]

- `import_unstructured` - CLI-only, no location/cluster hierarchy

[15.5174]

[3.17998]


**CLI-only:**
- `import file` - Single file import (use `skraak import file --db ... --path /path/to/file.wav`)
- `import_unstructured` - No location/cluster hierarchy

Replacement in CLAUDE.md at line 351 [13.363912]

B:BD[15.9463] → [15.9463:9514]

**Tools:** 11 total (read: 2, write: 4, import: 5)

[15.9463]

[2.423]

**Tools:** 10 MCP tools (read: 2, write: 4, import: 4) + 2 CLI-only import commands

Insertion in CHANGELOG.md at line 4 [17.1]

[17.81]

[2.506]


## [2026-02-21] Remove import_audio_file MCP Tool
**Breaking change:** Removed `import_audio_file` MCP tool. Use CLI command `skraak import file` for single file imports.

Insertion in CHANGELOG.md at line 9 [17.1]

[2.507]

**Rationale:** The MCP tool was redundant since:
1. Single file imports are better suited for CLI use (requires file path on local machine)
2. `import_audio_files` handles batch imports efficiently via MCP
3. Reduces MCP tool count from 11 to 10
**Changes:**
- **`cmd/mcp.go`** — Removed `import_audio_file` tool registration and adapter
- **`tools/import_file.go`** — Kept for CLI use only
- **`cmd/import.go`** — CLI command `skraak import file` unchanged
**Migration:** Use CLI command instead:
```bash
./skraak import file --db ./db/skraak.duckdb --dataset abc123 --location loc456 --cluster clust789 --path /path/to/file.wav
```
---

deleted the mcp tool import_file (1 file), beware as model did a run around

Dependencies

In channels

Change contents

Replacement in tools/import_file.go at line 56 [3.1]

Deletion in shell_scripts/test_import.sh at line 37 [6.16388]

Deletion in shell_scripts/test_import.sh at line 38 [6.16388]

Deletion in shell_scripts/test_import.sh at line 40 [6.16388]

Replacement in shell_scripts/test_import.sh at line 43 [6.16388]

Replacement in shell_scripts/test_import.sh at line 48 [6.16388]

Replacement in shell_scripts/test_import.sh at line 50 [6.16388]

Replacement in shell_scripts/TESTING.md at line 8 [10.57101]

Deletion in cmd/mcp.go at line 94 [4.1647]

Deletion in cmd/mcp.go at line 156 [4.1647]

Deletion in README.md at line 96 [13.334405]

Replacement in CLAUDE.md at line 119 [13.363912]

Replacement in CLAUDE.md at line 121 [13.363912]

Replacement in CLAUDE.md at line 124 [13.363912]

Deletion in CLAUDE.md at line 126 [13.363912]

Replacement in CLAUDE.md at line 128 [13.363912]

Replacement in CLAUDE.md at line 351 [13.363912]

Insertion in CHANGELOG.md at line 4 [17.1]

Insertion in CHANGELOG.md at line 9 [17.1]