Skip to content

test(sag): cover api client requests#6590

Open
gregkonush wants to merge 1 commit into
mainfrom
codex/sag-mp58j22c
Open

test(sag): cover api client requests#6590
gregkonush wants to merge 1 commit into
mainfrom
codex/sag-mp58j22c

Conversation

@gregkonush
Copy link
Copy Markdown
Member

Summary

  • Adds SAG API client coverage for AgentRun database-access request paths with URL encoding.
  • Verifies natural-language task submissions are sent as JSON POST requests.
  • Covers failed JSON response handling so server policy errors are surfaced to callers.

Related Issues

None

Testing

  • PASS: bun install --frozen-lockfile
  • PASS: bun run --filter @proompteng/sag test -- src/lib/sag-client.test.ts
  • PASS: bun run --filter @proompteng/sag test
  • PASS: bun run --filter @proompteng/sag tsc
  • PASS: bun run --filter @proompteng/sag lint
  • PASS: bun run --filter @proompteng/sag lint:oxlint
  • PASS: bun run --filter @proompteng/sag build

Screenshots (if applicable)

N/A

Breaking Changes

None

Checklist

  • Testing section documents the exact validation performed (or N/A with justification).
  • Screenshots and Breaking Changes sections are handled appropriately (removed or filled in).
  • Documentation, release notes, and follow-ups are updated or tracked.

@gregkonush
Copy link
Copy Markdown
Member Author

gregkonush commented May 14, 2026

Status: complete. PR is open, mergeable, and CI monitoring is complete.

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