fix(ops): accept list matches the backend; drawer reuses deriveGraphState; server count first - #57
Merged
Merged
Conversation
…tate; server count first - the file picker offered .docx/.xlsx (need the non-default kb-office build feature — a stock gateway rejects them AFTER the full transfer) and .css (never supported, 400 unsupported_file_type); dropped (plan 111 option (a)), with a comment naming src/kb/file/mod.rs as the source of truth; verified programmatically: every remaining entry appears in the backend lists - .heic added to IMAGE_EXTS (backend supports it); fixed the comment claiming images need no vision model — process_image posts to a chat-completions endpoint and requires a credential - the drawer's empty state now branches through deriveGraphState — the same pure function as the graph tab — instead of its own inline copy of the logic (the two surfaces disagreeing is how this started); three drawer branches pinned in the helper's test file - detail-view document count prefers group.document_count (correct since RantAIClaw plan 100) over the locally-fetched list length, which quietly hid a server-side divergence - re-extract toast wording was already fixed in #53 (plan 095 companion)
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.
Companion to RantAIClaw plans 095/097/100 — plan 111 (all three dependencies landed).
Items
.docx/.xlsxdropped (need the non-defaultkb-officefeature — a stock gateway rejects them after the full transfer);.cssdropped (never supported → 400). Option (a) per the plan's recommendation — option (b) would pulldocx-rs+calamineinto every build against the binary-size goal. Comment namessrc/kb/file/mod.rsas the source of truth..heicadded to images (backend supports it). Verified programmatically: every remaining frontend entry appears in the backend lists (empty diff both directions).process_imageposts to a chat-completions endpoint).deriveGraphState— the same pure function as the graph tab — replacing its inline copy from fix(ops): re-extract toast reports failed chunks with the reason #55; the two surfaces can no longer drift. Three drawer branches pinned in the helper test file.group.document_count(correct since RantAIClaw #460) over local list length, which quietly hid server-side divergence.Toast wording (step 2) was already shipped in #53.
Validation
vitest: 159/159 (3 new) ·next buildclean · list comparison scripted, both directions empty.Risk: picker/copy/branching only · Rollback: revert