mirror of
https://github.com/Comfy-Org/ComfyUI_frontend.git
synced 2026-05-12 00:42:03 +00:00
## Summary Fixed two minor documentation inaccuracies found during comprehensive documentation audit: - Corrected outdated "Lodash" reference to "Utility Functions" in unit testing guide - Updated package manager command from `npx` to `pnpm dlx` in Playwright skill documentation ## Changes Made ### Documentation Fixes #### docs/testing/unit-testing.md:150 - **Before**: `## Mocking Lodash Functions` - **After**: `## Mocking Utility Functions` - **Reason**: The section describes mocking `es-toolkit/compat` functions, not Lodash. The project uses es-toolkit as stated in AGENTS.md line 158 and docs/guidance/typescript.md line 60. #### .claude/skills/writing-playwright-tests/SKILL.md:117 - **Before**: `npx playwright show-trace trace.zip` - **After**: `pnpm dlx playwright show-trace trace.zip` - **Reason**: Project standardizes on pnpm, explicitly avoiding npx per AGENTS.md line 42: "use `pnpx` or `pnpm dlx` — never `npx`" ## Audit Summary Comprehensive audit verified accuracy of: - ✅ Core documentation (CLAUDE.md, AGENTS.md, README.md) - ✅ All docs/**/*.md files (40+ files including ADRs, testing guides, architecture docs) - ✅ All README files throughout repository (21 files) - ✅ All .claude/commands/*.md files (8 files) - ✅ Code examples and API references - ✅ File structure references (verified src/router.ts, src/i18n.ts, src/main.ts, config files exist) - ✅ Package dependencies (es-toolkit ^1.39.9 confirmed) - ✅ Script commands (pnpm test:unit, pnpm test:browser:local, etc.) - ✅ External resource links - ✅ ADR index and dates All other documentation remains accurate and up-to-date as of 2026-05-04. ## Review Notes This PR contains only two trivial corrections to terminology/commands. No functional changes, no code changes, no breaking changes. The documentation audit found the codebase documentation to be in excellent condition overall. Co-authored-by: christian-byrne <72887196+christian-byrne@users.noreply.github.com>