Skip to content

test: recover formatting utility coverage - #7995

Closed
KooshaPari wants to merge 3 commits into
cheaper-inference:fix/dependabot-audit-mainfrom
KooshaPari:agent/coverage-ratchet-recovery-20260721
Closed

KooshaPari wants to merge 3 commits into
cheaper-inference:fix/dependabot-audit-mainfrom
KooshaPari:agent/coverage-ratchet-recovery-20260721

Conversation

@KooshaPari

Copy link
Copy Markdown
Contributor

Summary

  • add focused coverage for stableAccountSuffix, fmtCompact, fmtFull, and truncateUrl
  • recover at least four function executions without changing production code or the quality baseline

Stack

This PR is intentionally stacked on #7953 (fix/dependabot-audit-main, d8af4e100b33003cfa8ba272510eec9d8d531197). Rebase/retarget onto main after #7953 lands.

Validation

  • Prettier check: passed
  • git diff --check: passed
  • focused test: not run locally because tsx is absent; no dependencies were installed
  • changed paths: exactly tests/unit/formatting-coverage-recovery.test.ts

No production, baseline, latency, workflow, Docker, Podman, WSL, self-hosted, or billed CI changes.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@KooshaPari
KooshaPari marked this pull request as ready for review July 21, 2026 12:35
@KooshaPari
KooshaPari requested a review from diegosouzapw as a code owner July 21, 2026 12:35
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@KooshaPari
KooshaPari force-pushed the agent/coverage-ratchet-recovery-20260721 branch 2 times, most recently from ad66d84 to eb1771c Compare July 21, 2026 13:38
@KooshaPari
KooshaPari force-pushed the agent/coverage-ratchet-recovery-20260721 branch from eb1771c to 0e770ea Compare July 21, 2026 14:09
@diegosouzapw

Copy link
Copy Markdown
Collaborator

This test file is identical to the one in your other PR #7996, but this PR's base is fix/dependabot-audit-main, which is actually my own open PR (#7953) rather than main or the release branch — so merging this one wouldn't land the content on any integration branch by itself. Since #7996 already carries the exact same test targeting main, I'll evaluate that one as the vehicle for this work and close this one as a duplicate once #7996 lands — let me know if you'd rather keep this one and retarget it instead.

@diegosouzapw
diegosouzapw deleted the branch cheaper-inference:fix/dependabot-audit-main July 23, 2026 00:24
@KooshaPari
KooshaPari deleted the agent/coverage-ratchet-recovery-20260721 branch August 13, 2026 06:52
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