Skip to content

test: fill SDK coverage gaps - #23

Merged
somus merged 1 commit into
mainfrom
somus/fill-test-coverage-gaps
Jun 16, 2026
Merged

test: fill SDK coverage gaps#23
somus merged 1 commit into
mainfrom
somus/fill-test-coverage-gaps

Conversation

@somus

@somus somus commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This pull request adds new test cases across several SDK packages to improve coverage of previously untested functionality. The new tests cover:

  • Core helpers: canonicalization of identity strings, stable UUID derivation with seed-part boundaries, and classification of credential key names, safe placeholders, and opaque token values.
  • Redaction configuration: validation of custom regex patterns, ensuring that assertSafeRegexSource correctly rejects unsupported constructs (backreferences, lookarounds, unsafe quantifiers) while allowing escaped syntax.
  • Redaction transform: entropy-based redaction behavior when explicitly enabled, and proper skipping of allowlisted secrets, opaque tokens, and already-redacted placeholders.
  • Render model: transcript summary and filtering logic aligned with ported viewer oracle cases, including tool grouping, semantic call‑id handling, and orphan result detection.
  • Sessions: discovery skipping of source references without paths, surfacing of health warnings, and propagation of reconcile passthrough warnings during session loading.
  • Store paths: resolution of the store root from explicit arguments, environment variables, or HOME, plus validation of content‑hash inputs.

All changes are test-only additions and do not modify production code.

@kody-ai

kody-ai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@ghost

ghost commented Jun 16, 2026

Copy link
Copy Markdown

Ready to review this PR? Stage has broken it down into 5 individual chapters for you:

Title
1 Test core identity and credential helpers
2 Add redaction entropy and regex-safety tests
3 Test store path resolution and environment fallbacks
4 Expand session discovery and reconciliation coverage
5 Port render-model oracle to committed tests
Open in Stage

Chapters generated by Stage for commit 3edb7dc on Jun 16, 2026 3:39am UTC.

@somus
somus merged commit 12b5741 into main Jun 16, 2026
2 checks passed
@somus
somus deleted the somus/fill-test-coverage-gaps branch June 16, 2026 03:48
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