Claude/agent issues cleanup 011 c us yj wb7 nmjya hfjx8 r pr#79
Closed
nonara wants to merge 5 commits intoclaude/phase2-parallel-fixes-011CUsYjWB7NMJYAHfjx8RPrfrom
Closed
Conversation
* chore: update dependencies to latest CJS-compatible versions Updated all dependencies to their latest CJS-compatible versions: Dependencies: - node-html-parser: 6.1.1 → 6.1.13 DevDependencies: - @types/jest: 28.1.1 → 29.5.14 (major) - @types/node: 18.11.5 → 18.19.130 - jest: 29.2.2 → 29.7.0 - ts-jest: 29.0.3 → 29.4.5 - ts-node: 10.9.1 → 10.9.2 - ts-patch: 2.0.2 → 3.3.0 (major) - typescript: 4.8.4 → 5.9.3 (major) Added yarn resolutions to force secure versions of transitive dependencies: - minimatch: ^3.1.0 (fixes ReDoS vulnerabilities) - brace-expansion: ^2.0.0 - shelljs: ^0.8.5 Security Impact: - Reduced vulnerabilities from 201 to 155 (23% reduction) - Production dependencies: 0 vulnerabilities ✓ - Remaining 155 are dev-only dependencies (acceptable) All tests pass, build successful, CJS compatibility maintained. * build(ci): Updated CI workflows to use latest versions --------- Co-authored-by: Claude <noreply@anthropic.com>
Moved nodeHtmlParserConfig from config.ts to utilities.ts to break the circular dependency where config.ts imported from utilities.ts and utilities.ts imported nodeHtmlParserConfig from config.ts. Resolves #74
Fixed transformer.js to correctly remove performance functions regardless of CI environment variable. The previous logic was: if (process.env.CI || !cfg.removePerf) return node; which would skip removal when CI=true. Now correctly checks only: if (!cfg.removePerf) return node; Also ensured ts-patch is properly installed so the transformer actually runs during compilation. Resolves #58
HTML is case-insensitive by spec, but the library was failing to process tags with mixed case (e.g., <Br>, <DIV>, <Strong>). This caused translation to stop prematurely, resulting in data loss. Root cause: The HTML parser with lowerCaseTagName: false would preserve the original case, but wouldn't recognize mixed-case void elements like <Br> as self-closing tags. This caused content after the tag to be incorrectly parsed as children of that tag. Solution: 1. Set lowerCaseTagName: true in nodeHtmlParserConfig to normalize all tags 2. Updated visitor.ts to handle tags case-insensitively using toUpperCase() 3. Added comprehensive tests for various mixed-case tag scenarios All translator lookups and element matching now work regardless of the original HTML tag casing, preventing data loss when processing HTML with inconsistent capitalization. Resolves #63
Addresses #69 and #66 by documenting expected behavior: - Explains paragraph spacing is standard markdown (blank lines between paragraphs) - Documents line breaks vs paragraphs behavior - Provides clear examples of maxConsecutiveNewlines option usage - Shows how to control consecutive newlines for different use cases Both issues are by-design behavior, not bugs. The maxConsecutiveNewlines option (default: 3) already provides the control users need.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.