Leidens algorithm and processmap#34
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
L1nusB
added a commit
to L1nusB/GitNexus
that referenced
this pull request
Mar 7, 2026
Cover duplicate-name collision, missing root and EACCES propagation, and lazy SKILL_NAMES population contract. Update refactor docs/test plan with new case IDs (abhigyanpatwari#34-abhigyanpatwari#37) and updated passing totals.
L1nusB
added a commit
to L1nusB/GitNexus
that referenced
this pull request
Mar 7, 2026
8 tests covering auto-discovery of skill names from the skills/ source directory: real source smoke test, prefix filtering, flat file discovery, directory-based discovery, mixed layouts, empty dir, and directories without SKILL.md. All 8 fail as expected — discoverSkillNames() is not yet exported from setup.ts. Existing 13 setup-skills tests remain green. feat(skills): auto-discover skill names from disk Replace hardcoded SKILL_NAMES array with discoverSkillNames() that reads the skills/ source directory at install time. This: - Automatically picks up gitnexus-pr-review (7 skills, up from 6) - Prevents breakage when skill files are renamed or added - Uses filesystem as source of truth instead of a hardcoded list Discovery supports flat files (*.md) and directory-based skills (*/SKILL.md), filtered to gitnexus-* prefix only. SKILL_NAMES export preserved (lazily populated) for backward compatibility. 870/870 unit tests passing. test(skills): add discovery edge-case coverage Cover duplicate-name collision, missing root and EACCES propagation, and lazy SKILL_NAMES population contract. Update refactor docs/test plan with new case IDs (abhigyanpatwari#34-abhigyanpatwari#37) and updated passing totals. fix(skills): address review feedback — guard cleanup, fix README count - Guard cleanupProjectLocalSkills on successful global install: don't remove local skills if no global replacement was actually installed - Update README skill count from 6 to 7, add gitnexus-pr-review entry - Stop swallowing non-ENOENT errors in installSkillsTo — only skip missing source skills, propagate real I/O failures 874/874 unit tests passing. chore: address review nitpicks — docs, unused variable - Update PR doc: mark cleanup guard edge case as fixed, clarify remaining tip visibility note - Update test plan: mark gap abhigyanpatwari#25 as resolved with guard reference - Remove unused dirSkillFile variable in installSkillsTo chore: cleanup
L1nusB
added a commit
to L1nusB/GitNexus
that referenced
this pull request
Mar 11, 2026
8 tests covering auto-discovery of skill names from the skills/ source directory: real source smoke test, prefix filtering, flat file discovery, directory-based discovery, mixed layouts, empty dir, and directories without SKILL.md. All 8 fail as expected — discoverSkillNames() is not yet exported from setup.ts. Existing 13 setup-skills tests remain green. feat(skills): auto-discover skill names from disk Replace hardcoded SKILL_NAMES array with discoverSkillNames() that reads the skills/ source directory at install time. This: - Automatically picks up gitnexus-pr-review (7 skills, up from 6) - Prevents breakage when skill files are renamed or added - Uses filesystem as source of truth instead of a hardcoded list Discovery supports flat files (*.md) and directory-based skills (*/SKILL.md), filtered to gitnexus-* prefix only. SKILL_NAMES export preserved (lazily populated) for backward compatibility. 870/870 unit tests passing. test(skills): add discovery edge-case coverage Cover duplicate-name collision, missing root and EACCES propagation, and lazy SKILL_NAMES population contract. Update refactor docs/test plan with new case IDs (abhigyanpatwari#34-abhigyanpatwari#37) and updated passing totals. fix(skills): address review feedback — guard cleanup, fix README count - Guard cleanupProjectLocalSkills on successful global install: don't remove local skills if no global replacement was actually installed - Update README skill count from 6 to 7, add gitnexus-pr-review entry - Stop swallowing non-ENOENT errors in installSkillsTo — only skip missing source skills, propagate real I/O failures 874/874 unit tests passing. chore: address review nitpicks — docs, unused variable - Update PR doc: mark cleanup guard edge case as fixed, clarify remaining tip visibility note - Update test plan: mark gap abhigyanpatwari#25 as resolved with guard reference - Remove unused dirSkillFile variable in installSkillsTo chore: cleanup
motolese
pushed a commit
to motolese/datamoto-gitnexus
that referenced
this pull request
Apr 23, 2026
…gorithm_and_processmap Leidens algorithm and processmap
zxs1633079383
pushed a commit
to zxs1633079383/GitNexus
that referenced
this pull request
Apr 30, 2026
根因: R-1 scaffold (deps.genE2ETests) 模板硬编码 Java 风格 (Test_xxx.java + JUnit). 之前 mattermost 仓 issue 拿到 Function:.go handler 时 S5 仍 照着模板出 Test_getScheduledPost.java 挂在 Go 仓上, 形成新错位 (虽然 handler 名是真的, 但 .java 后缀和 src/test/auto-generated/ 路径都不对). 修复: orchestrator S5 前按 handler filePath 后缀决策: - .java → 跑 deps.genE2ETests (现状) - 非 .java (.go/.rs/.ts/etc.) → 直接 skip with reason 落地后 cses (Java) 仓和 mattermost (Go) 仓的 PR 都不会再有错位文件. 真 trace 验证 (双仓 LIVE): - mm abhigyanpatwari#11 → MR !7: S5 skipped (filePath=*.go, R-1 仅支持 Java), pushed 1 file (仅报告无 Test_*.java) - cses abhigyanpatwari#34 → MR !36: S5 跑 (Test_view.java, Test_getConnection.java 真 handler 名), pushed 3 file (Java 仓产 java 测试合理)
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
1 participant
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.