-
Notifications
You must be signed in to change notification settings - Fork 12
docs: add Titan Paradigm Claude Code skills #473
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
5d29cd9
fix: prevent findDbPath from escaping git worktree boundary
carlos-alm deac23e
fix: address review — real git ceiling test, debug log, robust non-gi…
carlos-alm 34820e6
Merge remote-tracking branch 'origin/main' into fix/db-path-worktree-…
carlos-alm 83efde5
test: strengthen findRepoRoot and ceiling tests per review feedback
carlos-alm ccb8bd5
Revert "test: strengthen findRepoRoot and ceiling tests per review fe…
carlos-alm 73c51a4
Reapply "test: strengthen findRepoRoot and ceiling tests per review f…
carlos-alm e3c1d4a
fix: address Greptile review — flaky non-git test and misleading impo…
carlos-alm 5b4a5cd
merge: resolve conflict with main in db test imports
carlos-alm 20013d4
fix: resolve symlinks in findDbPath to fix ceiling check on macOS
carlos-alm 7c877d7
style: format connection.js try/catch block
carlos-alm 7e69570
Merge remote-tracking branch 'origin/main' into fix/db-path-worktree-…
carlos-alm 0d0fdd9
test: harden findDbPath fallback test to mock execFileSync
carlos-alm 2fa9dfc
fix: normalize findRepoRoot paths with realpathSync for cross-platfor…
carlos-alm 651c164
fix: use stat-based path comparison for ceiling check on Windows
carlos-alm 5dc707a
fix: remove test-only _resetRepoRootCache from public barrel export
carlos-alm 5a9dd2b
Merge branch 'main' into fix/db-path-worktree-boundary
carlos-alm b87a12b
fix: tighten test assertions and key repo root cache on cwd
carlos-alm 1301b9f
fix: normalize ceiling path in test to handle Windows 8.3 short names
carlos-alm 25e940b
Merge branch 'main' into fix/db-path-worktree-boundary
carlos-alm 5a77db1
fix: normalize ceiling path with realpathSync to handle Windows 8.3 s…
carlos-alm de87d18
Merge branch 'fix/db-path-worktree-boundary' of https://github.com/op…
carlos-alm 620128b
fix: normalize Windows 8.3 short paths in ceiling boundary test
carlos-alm e9e4de5
docs: add Titan Paradigm Claude Code skills for autonomous codebase c…
carlos-alm b3e46b5
merge: resolve conflicts with origin/main
carlos-alm a3ee610
fix: remove duplicate declarations in connection.js from merge
carlos-alm cc6dafd
docs: add /review skill for automated PR review sweeps
carlos-alm af0a536
Revert "docs: add /review skill for automated PR review sweeps"
carlos-alm 9ae497b
fix: resolve Windows CI test failure and doc inaccuracies
carlos-alm 26b8cfc
Merge branch 'main' into docs/titan-paradigm-skills
carlos-alm f73988a
fix: use dynamic snapshot deletion in titan-reset skill
carlos-alm 92be28c
fix: add merge-conflict guard to titan-gauntlet and titan-sync pre-fl…
carlos-alm 2f7bf1d
Merge remote-tracking branch 'origin/docs/titan-paradigm-skills' into…
carlos-alm 3a98d37
fix: correct installation cp paths in skills README
carlos-alm f2f9acf
Merge branch 'main' into docs/titan-paradigm-skills
carlos-alm 2bbb59d
Merge branch 'docs/titan-paradigm-skills' of https://github.com/optav…
carlos-alm 2a0ac94
fix: remove out-of-scope db test changes from docs PR
carlos-alm 6a3e6aa
Merge branch 'main' into docs/titan-paradigm-skills
carlos-alm 3211b8e
fix: correct NDJSON example threshold in titan-gauntlet skill
carlos-alm e80ef7c
fix: use cross-platform stdin in titan-reset snapshot deletion
carlos-alm 74a6a60
docs: add experimental warning to Claude Code skills README
carlos-alm 362e875
merge: resolve conflicts with main (keep PR improvements for skills, …
carlos-alm 1d75e02
fix: reset db.test.js to main (bad merge re-introduced tests without …
carlos-alm a4499a0
merge: resolve conflict in db.test.js by taking main's version
carlos-alm 00b5acc
Merge branch 'main' into docs/titan-paradigm-skills
carlos-alm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing merge-conflict guard in pre-flight
RECON's pre-flight explicitly says "If there are merge conflicts, stop and ask the user to resolve them", but GAUNTLET (and SYNC) silently proceed past a failed
--no-editmerge. A merge conflict will corrupt the working tree mid-audit.Consider adding the same guard used in RECON:
If the merge fails due to conflicts, stop: "Merge conflict detected. Resolve conflicts and re-run
/titan-gauntlet."There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed — added merge-conflict guard after the git merge step in both titan-gauntlet and titan-sync pre-flights (all 4 SKILL.md copies), consistent with RECON's existing guard.