Repository navigation
feat(skill): add time filters and memory lifecycle commands - #138
Merged
Merged
Conversation
added 4 commits
October 17, 2026 04:18
…-context Add since, until, time_order, owner_timezone, reference_time to recall and search. Add time filters and hypothetical_answer to recall-context. Enforce client-side fail-closed RFC 3339, time_order, and until>since validation. Pass through arguments via formal and temporary credential channels. Update documentation, command usage, golden snapshots, and bump skill to 1.1.42.
Add restore --id <id> --confirm [--reason <text>] to skills/xmemo with mandatory client-side confirmation, 404/403 mapping, documentation, and tests.
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.
Summary
Completes the approved XMemo Skill 1.1.42 scope for time filters, relationship listing, and soft-delete restore.
Changes
skills/xmemo/scripts/lib/time-filters.mjs.skills/xmemo/scripts/commands/memory-lifecycle.mjsso every Skill script and reference stays within the 12,288-byte cap. Command bodies were moved without changes.Verification
npm test: 664 passed, 0 failed, 0 skipped.npm run lint: passed.memory.mjsis 11,939 LF-normalized bytes.@xmemo/skill@1.1.42; staged CLI help loaded, both new modules had source-matching SHA-256 hashes, and both appeared innpm pack --dry-run.38964171685: 7/7 jobs passed across Node 20/22 on Ubuntu, Windows, and macOS plus the test job.xmemo.devlive gate passed on the 1.1.42 candidate after human-approved device login. No token was printed; logout remotely revoked the token and removed its isolated credential file.rememberPOST/v1/skill/operations: HTTP 200; row ID3e8e4e73-2a61-4f83-8f96-6335e76ab8cc; logicalmemory_id2d0a0e8d-73dc-402e-aab1-e764283a4685.searchGET/v1/memories/search, since2026-10-17T04:03:14Z: HTTP 200; 1 match.search, since2030-01-01T00:00:00Z: HTTP 200; 0 matches.relationshipsGET/v1/memories/{memory_id}/relationships: HTTP 200; 0 items.forget,restore, andread: HTTP 200 each; read returned the same active test memory.forget: HTTP 200; soft-deleted the test memory.logoutPOST/v1/auth/token/revoke-self: HTTP 200;remote_revoked=true,local_file_removed=true.Phase A boundary
This PR is review-ready work only. No merge, tag, or publish.