Skip to content

fix: address code review findings — version sync, CLAUDE.md, install.py gaps#177

Merged
Wool-xing merged 1 commit into
mainfrom
fix/code-review-findings-20260602
Jun 2, 2026
Merged

fix: address code review findings — version sync, CLAUDE.md, install.py gaps#177
Wool-xing merged 1 commit into
mainfrom
fix/code-review-findings-20260602

Conversation

@Wool-xing
Copy link
Copy Markdown
Owner

Summary

  • Fix ROADMAP version mismatch (V1.42.0 → V1.43.0)
  • Create CLAUDE.md with entry-point docs, directory map, architecture, design limitations
  • Fix README /smoke-test pseudo-command → correct workflow reference
  • Fix install.py: add missing FULL_GUIDE.md + tagent.yml.example to deployment
  • Fix install.py: remove /smoke-test from finish() message
  • Fix install.py: add memory/ directory creation
  • Create memory/.gitkeep

Closes code review issues #1 #2 #3 #5 #7 #9 #11

Test plan

  • grep "V1.42.0" ROADMAP.md — only historical changelog entries remain
  • CLAUDE.md exists with entry point explanation
  • README no longer references /smoke-test as a command
  • install.py copy_top_level_docs includes FULL_GUIDE.md and tagent.yml.example
  • install.py finish() references skills/smoke-test.md workflow, not /smoke-test
  • install.py create_dirs includes memory
  • memory/.gitkeep exists
  • All pre-commit hooks passed

…py gaps

- ROADMAP: V1.42.0 → V1.43.0 to match VERSION
- CLAUDE.md: new entry-point doc with directory map, design limitations
- README: replace /smoke-test pseudo-command with correct workflow reference
- install.py: add FULL_GUIDE.md + tagent.yml.example to copy_top_level_docs
- install.py: fix finish() to remove /smoke-test, reference skill doc instead
- install.py: add memory/ to create_dirs
- memory/.gitkeep: initialize required directory

Closes code review issues #1 #2 #3 #5 #7 #9 #11
@Wool-xing Wool-xing merged commit 50fc3af into main Jun 2, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant