feat: add traceable AI Discussions and IDD knowledge retrieval - #334
Open
kiki830621 wants to merge 4 commits into
Open
feat: add traceable AI Discussions and IDD knowledge retrieval#334kiki830621 wants to merge 4 commits into
kiki830621 wants to merge 4 commits into
Conversation
Add append-only publication, shared egress validation, retry reconciliation and bounded Discussion/comment/reply retrieval. Keep AI suggestions distinct from user decisions and expose incomplete evidence. Independent cross-model verification remains in progress; this commit does not claim verified status.
Refs #332 Replace guessed Markdown and URL exemptions with maintained parser ranges, reject NUL before shell decoding, and isolate body arguments. Normalize rendered newlines, retain snapshot titles, and validate API identities before marking writes posted. Full regression: 56 suites with no failures; targeted independent reviews passed. Six-lens model verification remains in progress.
Refs #331 Track decoded entity characters and zero-length deletions after code/URL exemptions, without joining fragments. Final scoped verification passed in disclosed Codex compatibility mode: five independent native reviewers and a separately governed Codex model; canonical Claude coverage was unavailable. Existing argument-value parsing behavior is tracked independently in #333.
This was referenced Sep 6, 2026
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
1 participant
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.
Selected AI conversations currently lose their decision history outside issue comments, and
idd-askcannot retrieve Discussions. This addsidd-discussto draft or explicitly publish append-only snapshots with original-message provenance, stable source IDs and conservative retry reconciliation, and extendsidd-askto read bounded Discussion/comment/reply evidence alongside its existing issue backend.The shared mention gate now uses maintained parser source ranges instead of delimiter/URL-prefix guesses. It rejects lossy NUL input and entity-obfuscated account characters, isolates body arguments, and preserves ambiguous contexts for scanning. Publisher newline/title handling and API response validation are also repaired.
Refs #331
Refs #332
Validation
gpt-5.6-sol/xhigh. Claude subprocesses timed out; this is explicitly not the canonical five-Claude-plus-Codex ensemble.Runtime and scope
Install the new shared-egress prerequisite with the same Python used by the helpers:
Missing or incompatible parsers refuse egress. Ambiguous table/HTML contexts are conservatively scanned. The repository must have Discussions enabled and a category selected for actual publication; the feature does not enable it automatically. It does not promise AI correctness, complete cross-platform histories or distributed exactly-once delivery.
The preexisting argument-value parsing mismatch is tracked separately in #333 and is not reachable through the publisher's fixed argument shape. This PR does not claim to resolve that separate issue.
Merge and issue closure remain manual IDD steps.