Fix audit follow-ups for usage evidence, teardown preservation, and status checks - #214
Merged
Merged
Conversation
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
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.
Usage could report incomplete OpenCode costs as complete totals, combine unrelated context measurements into a percentage, and allocate unbounded input buffers. Router teardown could overwrite post-setup user edits. Codex plugin checks also incorrectly rejected bundled capitalized skill names that Codex itself loads successfully.
This PR addresses those agentic-kit behaviors in eight focused commits:
Ruflo's CLI/MCP memory path divergence remains an upstream defect. This PR does not merge, delete, migrate, or globally redirect existing memory stores, and does not patch external plugins/packages. Fresh isolated confirmation is posted at ruvnet/ruflo#3196 (comment); follow-up integration is tracked in #213, which must remain open after this PR.
Validation:
pnpm test: 3,908 passed, six platform skips, zero failures; subsequent legacy suites passed. Coverage: 92.02% lines / 80.89% branches / 91.58% functions.pnpm run test:ui: 491 browser assertions and six suites passed before the final backend status/plugin changes; focused status/plugin regressions pass afterward.Evidence:
docs/audits/211-remediation.md,docs/audits/211-remediation-results.json, anddocs/audits/plugin-memory-status-followup.md.Refs #211, #212. Upstream integration follow-up: #213.