Error during validation: Unexpected token ...

[sQVe]

Hey, 👋🏼

I'm getting this issue fairly often:

● Update(src/providers/LinterProvider.test.ts)
  ⎿  Error: Edit operation blocked by hook:
     - Error during validation: Unexpected token 'T', "The TDD va"... is not valid JSON

The information is sadly quite truncated, so I cannot provide more information than the above. I would say that I get this issue around every 4-5 Write/Update call.

All the best!

[nizos]

Hi @sQVe,

That sounds really frustrating! Thanks for reporting it, I'll take a look. :)

[nizos]

Hi @sQVe,

Can you share the contents of the .claude/tdd-guard/data/ directory when you encounter this next time? It could prove useful for identifying and replicating the bug. Thanks again! :)

[sQVe]

Got it again now

● Update(src/linters/golangci/GolangciLint.ts)
  ⎿  Error: Edit operation blocked by hook:
     - Error during validation: Unexpected token 'T', "The TDD va"... is not 
     valid JSON

Sadly not much of interest in the data directory though.

==> .claude/tdd-guard/data/modifications.json <==
{
  "session_id": "0a9fb3c7-0985-402f-aac4-cda308dee5fa",
  "transcript_path": "/home/sqve/.claude/projects/-home-sqve-code-forks-tdd-guard/0a9fb3c7-0985-402f-aac4-cda308dee5fa.jsonl",
  "hook_event_name": "PreToolUse",
  "tool_name": "Edit",
  "tool_input": {
    "file_path": "/home/sqve/code/forks/tdd-guard/src/linters/golangci/GolangciLint.ts",
    "old_string": "import { LintResult } from '../../contracts/schemas/lintSchemas'\nimport { Linter } from '../Linter'\n\nexport class GolangciLint implements Linter {\n  async lint(filePaths: string[], configPath?: string): Promise<LintResult> {\n    const timestamp = new Date().toISOString()\n\n    const hasIssues = filePaths.some((file) =>\n      file.includes('file-with-issues.go')\n    )\n\n    if (hasIssues) {\n      return {\n        timestamp,\n        files: filePaths,\n        issues: [\n          {\n            file: filePaths[0],\n            line: 1,\n            column: 1,\n            severity: 'error' as const,\n            message: 'test issue',\n            rule: 'test-rule',\n          },\n        ],\n        errorCount: 1,\n        warningCount: 0,\n      }\n    }\n\n    return {\n      timestamp,\n      files: filePaths,\n      issues: [],\n      errorCount: 0,\n      warningCount: 0,\n    }\n  }\n}",
    "new_string": "import { LintResult, LintIssue } from '../../contracts/schemas/lintSchemas'\nimport { execFile } from 'child_process'\nimport { promisify } from 'util'\nimport { Linter } from '../Linter'\n\nconst execFileAsync = promisify(execFile)\n\nexport class GolangciLint implements Linter {\n  async lint(filePaths: string[], configPath?: string): Promise<LintResult> {\n    const timestamp = new Date().toISOString()\n    const args = buildArgs(filePaths, configPath)\n\n    try {\n      await execFileAsync('golangci-lint', args)\n      return createLintData(timestamp, filePaths, [])\n    } catch (error) {\n      if (!isExecError(error)) throw error\n\n      const results = parseResults(error.stdout)\n      return createLintData(timestamp, filePaths, results)\n    }\n  }\n}\n\nconst buildArgs = (files: string[], configPath?: string): string[] => {\n  const args = ['run', '--output.json.path=stdout']\n  if (configPath) {\n    args.push('--config', configPath)\n  } else {\n    args.push('--no-config')\n  }\n  args.push(...files)\n  return args\n}\n\nconst isExecError = (error: unknown): error is Error & { stdout?: string } =>\n  error !== null && typeof error === 'object' && 'stdout' in error\n\nconst parseResults = (stdout?: string): any[] => {\n  try {\n    if (!stdout) return []\n    const lines = stdout.split('\\n')\n    const jsonLine = lines[0]\n    if (!jsonLine.trim()) return []\n    const parsed = JSON.parse(jsonLine)\n    return parsed.Issues ?? []\n  } catch {\n    return []\n  }\n}\n\nconst createLintData = (\n  timestamp: string,\n  files: string[],\n  results: any[]\n): LintResult => {\n  const issues = extractIssues(results)\n  return {\n    timestamp,\n    files,\n    issues,\n    errorCount: countBySeverity(issues, 'error'),\n    warningCount: countBySeverity(issues, 'warning'),\n  }\n}\n\nconst extractIssues = (results: any[]): LintIssue[] =>\n  results.map(toIssue)\n\nconst toIssue = (issue: any): LintIssue => ({\n  file: issue.Pos?.Filename ?? '',\n  line: issue.Pos?.Line ?? 0,\n  column: issue.Pos?.Column ?? 0,\n  severity: 'error' as const,\n  message: issue.Text ?? '',\n  rule: issue.FromLinter,\n})\n\nconst countBySeverity = (\n  issues: LintIssue[],\n  severity: 'error' | 'warning'\n): number => issues.filter((i) => i.severity === severity).length"
  }
}
==> .claude/tdd-guard/data/test.json <==
{
  "testModules": [
    {
      "moduleId": "/home/sqve/code/forks/tdd-guard/src/linters/golangci/GolangciLint.test.ts",
      "tests": [
        {
          "name": "can be instantiated",
          "fullName": "GolangciLint > can be instantiated",
          "state": "passed"
        },
        {
          "name": "implements Linter interface with lint method",
          "fullName": "GolangciLint > implements Linter interface with lint method",
          "state": "passed"
        },
        {
          "name": "returns the file paths that were passed in",
          "fullName": "GolangciLint > returns the file paths that were passed in",
          "state": "passed"
        },
        {
          "name": "detects issues in file with lint problems",
          "fullName": "GolangciLint > detects issues in file with lint problems",
          "state": "failed",
          "errors": [
            {
              "message": "expected 0 to be greater than 0",
              "actual": "0",
              "expected": "0",
              "showDiff": false,
              "operator": "strictEqual",
              "name": "AssertionError",
              "ok": false,
              "stack": "AssertionError: expected 0 to be greater than 0\n    at /home/sqve/code/forks/tdd-guard/src/linters/golangci/GolangciLint.test.ts:38:34\n    at file:///home/sqve/code/forks/tdd-guard/node_modules/@vitest/runner/dist/chunk-hooks.js:752:20"
            }
          ]
        }
      ]
    }
  ],
  "unhandledErrors": [],
  "reason": "failed"
}

[sQVe]

And one more time.

  Update(src/linters/golangci/GolangciLint.test.ts)
  ⎿  Error: Edit operation blocked by hook:
     - Error during validation: Unexpected token 'T', "The analys"... is not valid JSON

==> .claude/tdd-guard/data/modifications.json <==
{
  "session_id": "0a9fb3c7-0985-402f-aac4-cda308dee5fa",
  "transcript_path": "/home/sqve/.claude/projects/-home-sqve-code-forks-tdd-guard/0a9fb3c7-0985-402f-aac4-cda308dee5fa.jsonl",
  "hook_event_name": "PreToolUse",
  "tool_name": "Edit",
  "tool_input": {
    "file_path": "/home/sqve/code/forks/tdd-guard/src/linters/golangci/GolangciLint.test.ts",
    "old_string": "  test('detects issues in file with lint problems', async () => {\n    const artifactsDir = join(process.cwd(), 'test', 'artifacts')\n    const configPath = join(artifactsDir, '.golangci.yml')\n    const filePath = join(artifactsDir, 'file-with-go-issues.go')\n\n    const result = await linter.lint([filePath], configPath)\n\n    expect(result.issues.length).toBeGreaterThan(0)\n    expect(result.errorCount).toBeGreaterThan(0)\n  })",
    "new_string": "  test('detects issues in file with lint problems', async () => {\n    const artifactsDir = join(process.cwd(), 'test', 'artifacts')\n    const configPath = join(artifactsDir, '.golangci.yml')\n    const filePath = join(artifactsDir, 'file-with-go-issues.go')\n\n    const result = await linter.lint([filePath], configPath)\n\n    expect(result.issues.length).toBeGreaterThan(0)\n    expect(result.errorCount).toBeGreaterThan(0)\n  })\n\n  test('returns actual golangci-lint output with correct structure', async () => {\n    const artifactsDir = join(process.cwd(), 'test', 'artifacts')\n    const filePath = join(artifactsDir, 'file-with-go-issues.go')\n\n    const result = await linter.lint([filePath])\n\n    expect(result.issues.length).toBe(3)\n    expect(result.issues[0].message).toBe('declared and not used: unused')\n    expect(result.issues[0].rule).toBe('typecheck')\n    expect(result.issues[0].file).toContain('file-with-go-issues.go')\n    expect(result.issues[0].line).toBe(6)\n    expect(result.issues[0].column).toBe(6)\n  })"
  }
}
==> .claude/tdd-guard/data/test.json <==
{
  "testModules": [
    {
      "moduleId": "/home/sqve/code/forks/tdd-guard/src/linters/golangci/GolangciLint.test.ts",
      "tests": [
        {
          "name": "can be instantiated",
          "fullName": "GolangciLint > can be instantiated",
          "state": "passed"
        },
        {
          "name": "implements Linter interface with lint method",
          "fullName": "GolangciLint > implements Linter interface with lint method",
          "state": "passed"
        },
        {
          "name": "returns the file paths that were passed in",
          "fullName": "GolangciLint > returns the file paths that were passed in",
          "state": "passed"
        },
        {
          "name": "detects issues in file with lint problems",
          "fullName": "GolangciLint > detects issues in file with lint problems",
          "state": "passed"
        }
      ]
    }
  ],
  "unhandledErrors": [],
  "reason": "passed"
}

[nizos]

Thank you!! This will help a lot!
I will keep you posted. :)