feat: implement delayed initialization strategy#371
Conversation
…ynchronous path checks
…kup migration for old configurations
Pull Request Title Suggestions
|
WalkthroughThis refactoring transitions workspace initialization from eager to lazy loading patterns. The changes remove immediate workspace initialization at module load time, replacing it with on-demand detection and initialization. A new dedicated workspace initialization module is introduced while removing the backward compatibility utils layer. The workspace detection logic is enhanced with better error handling, version migration support, and security improvements including remote name validation to prevent command injection. Changes
|
There was a problem hiding this comment.
Files selected (5)
- skills-entry/doc-smith/index.mjs (1)
- skills-entry/doc-smith/utils.mjs (1)
- skills-entry/doc-smith/workspace-init.mjs (1)
- utils/afs-factory.mjs (1)
- utils/workspace.mjs (7)
Files ignored due to filter (8)
- CLAUDE.md
- agents/bash-executor/ai/intent.md
- aigne.yaml
- package.json
- pnpm-lock.yaml
- skills-entry/doc-smith/ai/intent.md
- skills/doc-smith-auto-update/ai/intent.md
- skills/doc-smith-auto-update/ai/source-change-update-intent.md
Review comments generated (12)
- Review: 3
- LGTM: 9
Related Issue
https://team.arcblock.io/task/task/667380059203436544
Major Changes
pnpm-lock.yamlScreenshots
Test Plan
Checklist
Summary by AIGNE
Release Notes
Refactor:
New Feature:
Bug Fix: