Skip to content

fix: add remaining path discovery parity coverage#2845

Merged
code-yeongyu merged 9 commits intodevfrom
fix/path-discovery-parity-followup
Mar 26, 2026
Merged

fix: add remaining path discovery parity coverage#2845
code-yeongyu merged 9 commits intodevfrom
fix/path-discovery-parity-followup

Conversation

@code-yeongyu
Copy link
Copy Markdown
Owner

Summary

  • add regression coverage for ancestor project discovery across .claude/skills, .agents/skills, and .opencode/{skill,skills} bounded by the git worktree
  • add regression coverage for .agents skill awareness and for opencode project command discovery in both the command loader and slashcommand path
  • keep nested opencode command-loader names slash-separated so loader behavior matches the existing slashcommand discovery shape

Verification

  • bun test src/features/opencode-skill-loader/project-skill-discovery.test.ts src/features/claude-code-command-loader/loader.test.ts src/plugin-handlers/agent-config-handler-agents-skills.test.ts src/plugin-handlers/command-config-handler.test.ts src/tools/slashcommand/opencode-project-command-discovery.test.ts src/shared/opencode-command-dirs.test.ts
  • bun run typecheck
  • bun run build

Manual QA

  • verified ancestor discovery from a nested worktree path returns .agents and .opencode skills and exposes nested command name deploy/staging

code-yeongyu and others added 9 commits March 26, 2026 13:01
@code-yeongyu code-yeongyu merged commit 682eead into dev Mar 26, 2026
5 of 6 checks passed
@code-yeongyu code-yeongyu deleted the fix/path-discovery-parity-followup branch March 26, 2026 04:13
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 8 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Auto-approved: The PR adds comprehensive regression coverage for path discovery and ensures consistent slash-separated naming for nested commands, with no issues detected in review.

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