Review README for CLAUDE workflows repository#5
Merged
Conversation
Restructure README to follow a cleaner open-source format with: - Centered header with navigation links - Visual three-phase workflow diagram - Skills and commands in organized tables - Clear installation and usage instructions - Output structure documentation - Add MIT LICENSE file at project root
leeovery
deleted the
claude/review-readme-workflows-012azc3ab7C3wn5dt5VKb8pq
branch
leeovery
added a commit
that referenced
this pull request
Mar 5, 2026
- review-actions-loop: set status completed + pipeline check on clean synthesis and all-tasks-skipped paths (same bug class as R7 #5) - spec-review: work_type-aware source path resolution — bugfix specs use investigation paths, not discussion - convergence-awareness: remove redundant second prompt, use conclude language, remove legacy Discussion-ready marker - gather-context: remove Discussion-ready marker reference (status controls research inclusion now) - plan_id → ext_id in 3 discovery scripts, 3 tests, 2 SKILL.md docs, and implementation handoff template - technical-planning: tailor recovery instructions to actual files
leeovery
added a commit
that referenced
this pull request
Apr 20, 2026
…missing path Two related silent-failure modes rolled into one commit: MANIFEST_JS resolution (deferred #5): when neither candidate path existed, the const silently held a non-existent path. First manifest call ENOENT'd inside a silent try/catch and bulk operations returned empty. Now throws at module load with both candidate paths — an installation problem is surfaced immediately, not disguised as 'no artifacts to index'. Error swallowing (deferred #4): discoverArtifacts, getWorkUnitMeta, and the status unindexed-artifact block caught every execFileSync failure identically. 'Work unit not found' is expected (orphans), but corrupt JSON / bad work-unit name / missing manifest file were indistinguishable and silently skipped. Added isManifestKeyNotFound() to recognise expected misses, reportUnexpectedManifestError() to write everything else to stderr with the CLI's own message. Behaviour unchanged for the expected-miss path; real breakage is now visible. Closes deferred-issues #4, #5.
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.