Repository navigation
feat(diagnostics): add safe SDK support report (#13) - #216
Open
woahwhattheheck wants to merge 10 commits into
Open
woahwhattheheck wants to merge 10 commits into
woahwhattheheck wants to merge 10 commits into
Conversation
6 tasks
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.
Closes #13
/claim #13
Summary
Adds a deterministic, redaction-first SDK diagnostics report for support and debugging without introducing a live RPC side effect.
client.buildDiagnosticsReport(...)plus a standalone builder and public types;not-checkedunless a recent typedNetworkFailureis supplied, then reportsdegradedwith the existing safe recovery diagnostic;NetworkFailurecannot inject an arbitrary raw message into support output;docs/diagnostics-report.md;Acceptance traceability
src/diagnostics/report.ts+ public exports +AegisClient.buildDiagnosticsReportconfiguration.rpc,configuration.network,configuration.contract, plus conservativenetwork.statuscompliance.module,protocolState: not-checked,legalStatus: not-assessedtests/diagnostics-report.test.tshas four focused cases covering config redaction, preset semantics, typed-error redaction, and five-error bounddocs/diagnostics-report.md, README section, API referenceValidation truth
Source branch was created from sponsor
mainat69fff2c7e8c6fe801428fc1bb71d064c5c94949c; final pre-publication fence confirmed sponsormainstill equals that SHA and issue #13 still had no PR cross-reference or @woah claim.Connector readback reports 8 changed paths and branch head
3f7ced355d0f299dcde4623d209daf593ca72ccb, 10 commits ahead / 0 behind the base. A focused static source audit confirmed the report does not directly copyrpcUrl,networkPassphrase,contractId, orerror.message, the client method/export are present, and the diagnostics doc contains both network and compliance boundaries.The focused Jest cases were authored but not executed in this session because the cloud container could not resolve GitHub to materialize dependencies/source. No test-suite, build, CI, or runtime green result is claimed.
Compensation
I, @woahwhattheheck (GitHub ID 293286387), am the original contributor for this issue-scoped implementation and affirmatively request sponsor review, contributor credit, and any eligible GrantFox OSS / Official Campaign FWC26 compensation for issue #13. Please confirm assignment/eligibility, accepted scope, payable amount, and settlement route if applicable. The
Maybe Rewardedlabel is conditional; no award or payment is asserted.