Replace process-file.js with block-replacer package
Replace process-file.js in core with the new block-replacer package as requested in issue #100.
Changes
- Updated core package.json to include comment-block-replacer workspace dependency
- Updated import in index.js to use new package instead of local process-file.js
- Removed old process-file.js file
Notes
- Maintains exact same API and functionality
- No breaking changes
Fixes #100
Generated with Claude Code
ok tests work and it looks good.
Can you start addressing the type errors in /core?
Run npm run types in packages/core to see the current type errors
Follow these rules:
- We want to keep all types in JSDoc
- We want to import type defs from deps and reuse if possible like @typedef {import('../src').ProcessFileOptions} ProcessFileOptions
- Don't add any .ts or .d.ts files this should all be with jsdocs
Run npm run types in the other packages to verify they work too if you make any updates to the deps
@claude
I'll help you address the type errors in the /core package using JSDoc annotations.
Todo List
- [x] Read CLAUDE.md for repository setup instructions
- [x] Install dependencies and run initial type check
- [x] Analyze current type errors in packages/core
- [ ] Fix JSDoc syntax errors in src/index.js
- [ ] Fix JSDoc return type annotations in src/utils/text.js
- [ ] Import types from dependencies where possible
- [ ] Verify other packages still pass type checks
- [ ] Commit fixes to the PR branch
Analysis Complete
Found several JSDoc annotation issues in the core package:
- src/index.js: Invalid JSDoc comment format and non-standard syntax
- src/utils/text.js: Missing return type annotations with proper curly braces
Working on fixes now...
β Claude Code execution failed
An error occurred while processing Pull Request #101.
π Error Info: Pull Request | issue_comment | @DavidWells
π Detailed Log: Actions run log
π€ Possible Causes and Solutions:
1. Temporary Issue (most common)
- Temporary limitations of the Claude API
- β Solution: Rerun with
claudeafter 5 minutes
2. Timeout (15-minute limit)
- The task may be too complex
- β Solution: Break down the task with more specific instructions
3. Configuration Issue
- Expired or misconfigured tokens
- β Solution: Check in Settings β Secrets β Actions
CLAUDE_ACCESS_TOKENCLAUDE_REFRESH_TOKENCLAUDE_EXPIRES_AT
π‘ Rerun Tips:
- Be specific:
claude review src/components/Button.tsx - Step by step: Start with one file
- Be clear: State the expected outcome
π Rerun: Please try again with claude [specific instructions]
π Support: If the problem persists, please contact an administrator
@claude try again. Limit the scope and try to help some of the errors. its okay if you finish before you fix all of them
Claude finished @DavidWells's task ββ View job
I'll help you fix the type errors in the /core package using JSDoc annotations.
Todo List
- [x] Read CLAUDE.md for repository setup instructions
- [ ] Install dependencies and run initial type check (need Bash permissions)
- [x] Analyze current type errors in packages/core
- [x] Fix JSDoc syntax errors in src/index.js
- [x] Fix JSDoc return type annotations in src/utils/text.js
- [x] Import types from dependencies where possible
- [ ] Verify other packages still pass type checks
- [x] Commit fixes to the PR branch
Fixes Applied
I've successfully fixed several JSDoc annotation issues:
-
src/index.js:
- Fixed invalid JSDoc comment format (
/**!β/**) - Fixed non-standard inline JSDoc syntax
- Added import for
ProcessFileOptionstype fromcomment-block-replacer
- Fixed invalid JSDoc comment format (
-
src/utils/text.js:
- Added proper JSDoc with return types to 10+ functions
- Fixed return type format (
@returns stringβ@returns {string}) - Imported
SyntaxTypefrom local types module
-
src/utils/index.js:
- Added JSDoc to all utility functions
-
src/utils/fs.js:
- Added JSDoc to key utility functions
Next Steps
To complete the type checking:
- I need Bash permissions to run
npm run typesand see remaining errors - Once I can run the type checker, I can identify and fix any remaining issues
- Verify other packages in the monorepo still pass their type checks
The fixes I've made should resolve many of the JSDoc syntax errors. However, I wasn't able to run the type checker to see the full list of remaining issues due to missing Bash permissions.
π¬ Claude Code execution complete
π Execution Info: Pull Request #101 | Unknown (93 files) | @DavidWells
πΏ Branch: claude/issue-100-20250624_065004 β master
π Automated management executed:
π‘ Example commands for Claude:
π Analysis & Review:
claude Please review the codeclaude Please perform a security checkclaude Please suggest performance improvements
π οΈ Tasks & Implementation:
claude Please add test casesclaude Please fix this issue and create a PRclaude Please suggest refactoring
π Management & Operations:
claude Please create a releaseclaude Please check security alertsclaude Please optimize the workflow
π PR Specific:
claude Please perform a final check before mergingclaude Please check for breaking changes
π Rerun: You can run again anytime with claude [your instructions]