Skip to content

Add exact MMS file-transfer failure diagnostics#39

Merged
masarray merged 3 commits into
mainfrom
fix/visible-file-transfer-diagnostics
Jul 20, 2026
Merged

Add exact MMS file-transfer failure diagnostics#39
masarray merged 3 commits into
mainfrom
fix/visible-file-transfer-diagnostics

Conversation

@masarray

Copy link
Copy Markdown
Owner

Field problem

Fault-record discovery succeeds and the remote CFG/DAT/INF bundle is complete, but Download still ends in a generic InvalidDataException. The application currently loses the primary failing request because the best-effort FileClose can overwrite the shared last-request/last-response fields.

Changes

  • capture a bounded per-transfer protocol timeline
  • preserve the first/primary failure even when cleanup also fails
  • report stage, remote path, invoke ID, signed FRSM, FileRead number, bytes transferred, moreFollows, exception type/message, association/pump state, pending confirmed operations, receive routing, pump fault, and compact request/response hex
  • distinguish FileOpen, each FileRead, local write/flush, validation, FileClose, timeout, and skipped-close failures
  • embed the full trace in fault-record bundle failure results
  • keep the trace bounded to avoid retaining file payloads or unbounded history

This instrumentation is designed to make the next field result decisive before any packet-capture comparison is required.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@masarray
masarray merged commit d51b04e into main Jul 20, 2026
1 check passed
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