fix: use node: protocol for imports

[codewithkenzo]

Changes:

• Use node: protocol for builtin modules (util, fs, path, url)
• Prefix unused variables with underscore

Scout-detected fixes.

Greptile Overview

Greptile Summary

This PR modernizes Node.js imports using the node: protocol and prefixes unused variables with underscores, but introduces a critical bug in src/commands/api.ts.

Critical Issue:

• src/commands/api.ts:32 - The parseTypeScript import was accidentally removed, causing ReferenceError: parseTypeScript is not defined at runtime when running the api command

Safe Changes:

• All node: protocol additions for built-in modules (util, fs, path, url) are safe and follow modern Node.js best practices
• Optional chaining improvement in typescript.ts:47 maintains equivalent logic
• Prefixing unused variables (_commentContent, _error) follows linting conventions correctly

Confidence Score: 1/5

• This PR is blocked - it breaks the api command with a runtime error
• The missing import in src/commands/api.ts causes immediate runtime failure. While other changes are safe, this critical bug makes the PR unmergeable
• src/commands/api.ts must restore the parseTypeScript import before merging

Important Files Changed

Filename	Overview
src/cli.ts	Updated import to use node: protocol for built-in util module - safe modernization
src/commands/api.ts	Updated imports to use node: protocol but accidentally removed parseTypeScript import, causing runtime error on line 32
src/parsers/typescript.ts	Updated imports to use node: protocol, improved optional chaining, and prefixed unused variables with underscore - all changes safe

Sequence Diagram

sequenceDiagram
    participant CLI as cli.ts
    participant API as commands/api.ts
    participant Parser as parsers/typescript.ts
    participant FS as node:fs/path/url
    
    CLI->>CLI: Import parseArgs from node:util ✓
    CLI->>API: Call generateApiDocs()
    API->>API: Import from node:fs/path/url ✓
    API->>API: ❌ Missing parseTypeScript import
    API->>Parser: Call parseTypeScript() 
    Note over API,Parser: ReferenceError thrown here
    Parser->>Parser: Import from node:fs/path ✓
    Parser->>FS: File operations
    FS-->>Parser: Return data
    Parser-->>API: Return modules
    API-->>CLI: Complete

Loading

[greptile-apps]

_{3 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Removed import breaks parseTypeScript call on line 32. This causes ReferenceError: parseTypeScript is not defined at runtime.

Suggested change

	import { loadConfig } from '../config.js';
	import { loadConfig } from '../config.js';
	import { parseTypeScript } from '../parsers/typescript.js';

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/commands/api.ts
Line: 5:5

Comment:
Removed import breaks `parseTypeScript` call on line 32. This causes `ReferenceError: parseTypeScript is not defined` at runtime.

```suggestion
import { loadConfig } from '../config.js';
import { parseTypeScript } from '../parsers/typescript.js';
```

How can I resolve this? If you propose a fix, please make it concise.