Skip to content

fix: bounded Responses replay refusal diagnostics - #449

Merged
cloga merged 2 commits into
mainfrom
cloga-responses-replay-diagnostics
Oct 25, 2026
Merged

cloga merged 2 commits into
mainfrom
cloga-responses-replay-diagnostics

Conversation

@cloga

@cloga cloga commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Summary

Fixes #446. Carries forward the independent diagnostic value of closed draft #218 against the current pinned pi-ai 1.1.0 / DSH 0.2.1-rc.3, rather than cherry-picking its obsolete implementation. #218 was closed as migration/handoff, not as an implementation-completion claim; its original branch remains preserved.

Adds fixed structural refusal codes, zero-based input positions, allowlisted type/status and field-presence booleans to the existing message-only native INVALID_REQUEST surface. Diagnostics exclude IDs, summaries, tool arguments/results, unknown discriminator values, and encrypted bytes. Existing replay acceptance, recovery consent, cancellation/account isolation, and stored-history ownership remain unchanged.

Adds hostile-input regressions, actual pinned-SDK synthetic two-turn SSE rejection and terminal-encryption-backfill controls, managed provider/route assertions, and matching English/Chinese documentation. Prepares same-Core 0.1.0-alpha.4.dsh.0.2.1.rc.3 metadata required by unchanged release policy, retaining alpha.3 history. No dependency, lockfile, CI policy, or Core changes.

Validation

Authoritative exact-head CI passed at 973052e82a951ae19c8257a0473200ca718d0b19: https://github.com/cloga/dsh-github-copilot/actions/runs/40186158312

  • Current base: main at edca08c65d6b7682aa5b5bdc6b4edf104ac9f198 (includes merged feat(tooling): enforce explicit environments and exact-head CI evidence #452 and test: verify packed plugin installation and pinned-host loading #450); rebase completed without conflicts.
  • All five required checks passed: verify Windows/Linux, published adapter Windows/Linux, and release-ready. Release job was skipped because this is a PR.
  • Full gates passed on Windows/Linux: frozen install, exact tagged Core fixtures, strict source/test typechecks, pnpm verify, build, all tests, package smoke, pack/tarball verification, and the merged installation smoke through the pinned Core host.
  • Windows full suite: 132 files, 2796 passed and 2 existing skips; tooling suite: 550 passed. Tagged-source suite: 467 passed, 2 existing skips; all 28 native reviewer regressions executed. Relevant suites: replay compatibility 177, pinned SDK replay 2, provider 97, route 183.
  • Published-artifact typechecks and native adapter/Core fixtures passed on both platforms. Release metadata policy passed for alpha.4.

Local results are auxiliary; GitHub CI is authoritative. No real credentials or live model calls were used. Synthetic fixtures do not establish live provider acceptance.

Scope and delivery

Plugin-only change. No Core patch, new retry, history deletion, automatic model switch, or installation-tool changes. This PR is ready for review, but has not been merged. Merge, tag, publication, profile installation, restart, and live model calls are not authorized by this task.

cloga and others added 2 commits October 25, 2026 18:46
Carry forward safe structural refusal evidence and current pinned SDK regression coverage without changing replay acceptance or durable history.

Assisted-by: GitHub Copilot

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Align required same-Core candidate metadata without authorizing tag, merge, publication or installation.

Assisted-by: GitHub Copilot

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@cloga
cloga force-pushed the cloga-responses-replay-diagnostics branch from ba9738a to 973052e Compare October 25, 2026 10:46
@cloga
cloga marked this pull request as ready for review October 25, 2026 10:54
@cloga
cloga merged commit 6c36189 into main Oct 25, 2026
6 checks passed
@cloga
cloga deleted the cloga-responses-replay-diagnostics branch October 25, 2026 10:59
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.

Carry forward safe Responses replay diagnostics on current SDK

1 participant