@lorekit/cli 1.46.0 → 1.47.0

package/README.md

@@ -466,9 +466,9 @@ that lesson's detail sheet. It sets **both** the `lesson` param (which opens the
 sheet) and `scope` — not because scope is needed to find the lesson (the sidebar
 reads one unfiltered recent set), but so the Explorer list *behind* the sheet is
 filtered to the lesson's own scope. Filter flags mirror the Explorer: `--q`
-(search), `--owner <all|personal|orgId>`, `--tags <a,b,c>` (label filter, AND
+(search), `--owner <all|personal|org-slug>`, `--tags <a,b,c>` (label filter, AND
 across labels; comma-separated or a JSON array), `--range`/`--from`/`--to`,
-`--archived`, `--view <scope|time>`.
+`--archived`.
 
 Every param is `encodeURIComponent(JSON.stringify(value))` — the exact inverse of
 how the dashboard's `useUrlState` reads it back (`JSON.parse`, falling back to the

package/bin/lorekit.mjs

@@ -88,7 +88,7 @@ ${c.bold('Commands')}
   link (url)  Print a shareable dashboard deep-link URL for the current context,
               a scope, or a specific lesson (opens its detail sheet). No args
               links to the cwd's most-specific scope. Filter flags mirror the
-              Explorer (--q / --owner / --tags / --range / --archived / --view);
+              Explorer (--q / --owner / --tags / --range / --archived);
               --base or LOREKIT_APP_URL override the dashboard host. --json. Pipe it:
               lorekit link | pbcopy.
   bootstrap   Apply the BYOD schema to a user-supplied Supabase database.
@@ -585,13 +585,12 @@ ${c.bold('Options')}
       --key <key>         Name the key explicitly — the way to link to a key
                           that itself contains \`::\`
       --q <text>          Pre-fill the Explorer search box
-      --owner <o>         Ownership filter: all | personal | <orgId>
+      --owner <o>         Ownership filter: all | personal | <org-slug>
       --tags <a,b,c>      Label filter (AND across labels); comma-separated or a JSON array
       --range <json>      Date range as {"from":"YYYY-MM-DD","to":"YYYY-MM-DD"}
       --from <date>       Range start (shorthand for --range)
       --to <date>         Range end (shorthand for --range)
       --archived          Include archived memories
-      --view <mode>       Explorer view: scope | time
       --base <url>        Dashboard base URL (else LOREKIT_APP_URL, default https://lorekit.io)
       --json              Machine-readable { url, surface, base, params }
 
@@ -658,7 +657,13 @@ const KNOWN_FLAGS = [
   'from', 'to', 'apply', 'yes', 'hooks', 'no-hooks', 'mcp-json', 'force', 'deep', 'adapter',
   'event', 'json', 'scope', 'key', 'threshold', 'help', 'version', 'telemetry',
   'value', 'tags', 'source-agent', 'trigger', 'kind', 'host', 'ttl-days', 'clear-ttl', 'org', 'remote', 'local',
-  'link', 'base', 'q', 'owner', 'range', 'view', 'archived',
+  // `view` is accepted-and-IGNORED, not documented: the Explorer dropped the
+  // scope/time tab so the flag is a no-op, but `link` is a HUMAN_COMMAND that
+  // rejects UNKNOWN options — so keeping `view` listed for a release stops an
+  // existing `lorekit link --view time` from hard-failing on an unknown-option
+  // error. It is parsed and discarded (nothing reads `args.view`). Remove it once
+  // 1.x links have aged out.
+  'link', 'base', 'q', 'owner', 'range', 'archived', 'view',
   'origin-repo', 'origin-branch', 'origin-commit', 'origin-pr', 'no-origin',
   // Scale-aware survey flags
   'all', 'max', 'since', 'until', 'key-prefix', 'cluster-by-key',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lorekit/cli",
-  "version": "1.46.0",
+  "version": "1.47.0",
   "description": "Install the LoreKit shared-memory skill and run health checks for the LoreKit MCP server.",
   "license": "MIT",
   "repository": {

package/src/deeplink-pure.mjs

@@ -36,7 +36,7 @@ export const LORE_PARAM_DEFAULTS = {
   // accepts ISO instants in that arm and a relative { preset: '7d' } arm
   // (packages/web/src/lib/time-range.ts); neither has a CLI flag yet.
   range: null,
-  owner: 'all', // 'all' | 'personal' | { orgId }
+  owner: 'all', // 'all' | 'personal' | '<org-slug>' — folds into an owner filter (00064)
   // Filter[] | null — the Explorer's multi-dimension filter bar (label / agent /
   // trigger / repo / branch / pr). `null`, NOT `[]`, is the default on purpose:
   // the app has to tell "the param is absent" from "the bar is explicitly
@@ -46,7 +46,6 @@ export const LORE_PARAM_DEFAULTS = {
   // "unfiltered".
   filters: null,
   tags: [], // string[] — legacy label filter (AND across labels); [] means "no filter". Still READ by the app, superseded by `filters`
-  view: 'scope', // 'scope' | 'time'
   // 'active' | 'archived' | 'expiring' | null — the Explorer's Status control.
   // `null`, NOT 'active', is the default for `filters`' reason: the app has to
   // tell "absent" from an explicit choice, because an absent `status` falls back
@@ -60,9 +59,9 @@ export const LORE_PARAM_DEFAULTS = {
 
 // A stable, readable param order (also makes URLs deterministic for tests).
 // Mirrors the `useUrlState` call order in `LoreExplorer.tsx` (+ the `lesson`
-// param last), so `filters` and `tags` sit between `owner` and `view`. `scope`
+// param last), so `filters` and `tags` sit between `owner` and `status`. `scope`
 // precedes `lesson` so a lesson link reads `?scope=…&lesson=…`.
-const PARAM_ORDER = ['scope', 'q', 'range', 'owner', 'filters', 'tags', 'view', 'status', 'archived', 'lesson'];
+const PARAM_ORDER = ['scope', 'q', 'range', 'owner', 'filters', 'tags', 'status', 'archived', 'lesson'];
 
 // Strip trailing slashes from a base URL, falling back to the default when the
 // input is empty/absent. Pure.
@@ -148,18 +147,18 @@ export { resolveScopeArg, resolveScopeKeyArgs, isScopeString, scopeIssue } from
 
 // ── Flag → param coercion (pure, shared by the `link` command) ────────────────
 
-// Coerce the `--owner` flag to an `OwnerFilter`: 'all' (default) / 'personal' /
-// any other non-empty string → `{ orgId }`. Pure.
+// Coerce the `--owner` flag to the legacy `owner` param value: `personal`, an
+// org SLUG, or `all` (default → omitted). Returns the STRING the app folds into
+// an owner filter (`filtersFromLegacyOwner`) when the `filters` param is absent
+// — which is why the CLI keeps writing the legacy param rather than `filters`:
+// `owner` and the legacy `tags` param fold together, so `--owner acme --tags
+// perf` yields BOTH, whereas a `filters` param would make the app ignore the
+// legacy tags. NOT the old `{orgId}` OBJECT: the owner facet keys on the SLUG,
+// and the app cannot resolve a uuid to a slug in this pure path, so the object
+// form silently dropped the filter it named (00064). A slug lands verbatim. Pure.
 export function parseOwnerArg(owner) {
-  if (typeof owner !== 'string' || !owner || owner === 'all') return 'all';
-  if (owner === 'personal') return 'personal';
-  return { orgId: owner };
-}
-
-// Coerce the `--view` flag to a `ViewMode`: only 'time' is non-default; anything
-// else (incl. absent/invalid) → 'scope'. Pure.
-export function parseViewArg(view) {
-  return view === 'time' ? 'time' : 'scope';
+  if (typeof owner !== 'string' || !owner) return 'all';
+  return owner;
 }
 
 // Coerce the `--tags` flag to a normalized `string[]` label filter, mirroring the

package/src/link.mjs

@@ -22,7 +22,6 @@ import {
   buildLoreUrl,
   mostSpecificScope,
   parseOwnerArg,
-  parseViewArg,
   parseRangeArg,
   parseTagsArg,
   resolveScopeKeyArgs,
@@ -55,7 +54,6 @@ export async function link(args) {
   // Filter flags (all optional; each JSON-encoded + default-omitted downstream).
   const q = typeof args.q === 'string' ? args.q : '';
   const owner = parseOwnerArg(args.owner);
-  const view = parseViewArg(args.view);
   const range = parseRangeArg(args);
   const tags = parseTagsArg(args.tags);
   const archived = Boolean(args.archived);
@@ -65,7 +63,6 @@ export async function link(args) {
     (typeof args.scope === 'string' && Boolean(args.scope)) ||
     Boolean(q) ||
     owner !== 'all' ||
-    view !== 'scope' ||
     range !== null ||
     tags.length > 0 ||
     archived;
@@ -84,7 +81,6 @@ export async function link(args) {
   if (q) params.q = q;
   if (owner !== 'all') params.owner = owner;
   if (tags.length) params.tags = tags;
-  if (view !== 'scope') params.view = view;
   if (range !== null) params.range = range;
   if (archived) params.archived = true;