Skip to content

test(rules): improve coverage for rule loading#66

Merged
phantom5099 merged 1 commit into
mainfrom
fork-pr-542-1777697594
May 2, 2026
Merged

test(rules): improve coverage for rule loading#66
phantom5099 merged 1 commit into
mainfrom
fork-pr-542-1777697594

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented May 2, 2026

Requested by @phantom5099

Summary

  • add isolated rules/context tests so rule prompt tests no longer depend on the reviewer machine home directory
  • cover canceled-context, missing-file and read-error branches in the new rules load/store APIs
  • normalize relative project roots to absolute paths so project rule read/write stays round-trippable

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>
@phantom5099 phantom5099 merged commit 28328b3 into main May 2, 2026
2 checks passed
@fennoai fennoai Bot deleted the fork-pr-542-1777697594 branch May 2, 2026 06:23
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.

2 participants