Address CodeRabbit review comments on CLI v2 PR

[Copilot]

Addresses all unresolved CodeRabbit review comments from PR #73 (CLI v2 branch), fixing critical and major issues identified during code review.

Changes Made

CI/CD Improvements

• Changed npm install to npm ci in workflow files (auto-dev-release.yml, npm-test.yaml) for deterministic, reproducible builds
• Added CI-only execution guard to bump-sync-version-core.js to prevent accidental local execution of destructive git operations

Bug Fixes

• Fixed createDefaultStep() call to include required stepType parameter (defaulting to 'goTo')
• Replaced Node.js-incompatible requestAnimationFrame with setImmediate in CLI runner
• Fixed aiConfig initialization in refineStep.js to properly read from config.ai

Code Quality

• Fixed log function signature from (config, level, message) to natural order (message, level, config) and updated all call sites
• Standardized OpenAI integration property naming to openAi (camelCase) throughout the codebase

Schema Validation

• Added required fields array to sourceLocation_v3.schema.json
• Added conditional validation requiring commentFormat when isInline is true
• Fixed example Support DITA input files #2 commentFormat from htmlComment to linkReference

Issues Addressed

All 10 CodeRabbit review comments resolved:

• 3 Critical severity issues (empty step creation, log parameter order, property naming inconsistency)
• 2 Major severity issues (CI reproducibility, Node.js compatibility)
• 5 Additional issues (CI guard, dead code, schema validation)

The changes improve code quality, reliability, and correctness while maintaining backward compatibility.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

Note

Free review on us!

CodeRabbit is offering free reviews until Wed Dec 17 2025 to showcase some of the refinements we've made.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[hawkeyexl]

@copilot All unresolved comments left by CodeRabbit.

[Copilot]

@copilot All unresolved comments left by CodeRabbit.

All unresolved CodeRabbit comments have been addressed in commits 21467ae and fa2e1e2:

Fixes implemented:

1. ✅ Changed npm install → npm ci in CI workflows for deterministic builds
2. ✅ Added CI-only execution guard to bump-sync-version-core.js
3. ✅ Fixed createDefaultStep('goTo') to include required stepType parameter
4. ✅ Replaced requestAnimationFrame with Node.js-compatible setImmediate
5. ✅ Fixed log function signature and all call sites to use natural parameter order
6. ✅ Standardized OpenAI property naming to openAi throughout codebase
7. ✅ Fixed aiConfig initialization to actually read from config.ai
8. ✅ Added required fields and conditional validation to sourceLocation schema

All 10 critical and major issues from the CodeRabbit review are now resolved.