Skip to content

docs(skill): add share link intent routing to recall skill#57

Merged
samzong merged 1 commit into
mainfrom
docs/recall-share-routing
Jun 24, 2026
Merged

docs(skill): add share link intent routing to recall skill#57
samzong merged 1 commit into
mainfrom
docs/recall-share-routing

Conversation

@samzong

@samzong samzong commented Jun 24, 2026

Copy link
Copy Markdown
Owner

What's changed?

  • Add intent routing table to map share, refresh, resume, search, and project-review prompts to the right workflow
  • Add a dedicated "Publish Or Refresh Share Link" section with step-by-step CLI commands
  • Expand skill triggers for Chinese and English share/update phrases, plus Grok as a supported source
  • Demote --dry-run to preview-only in command examples and anti-patterns

Why

  • Agents were stopping at recall session share --dry-run, which returns a URL locally but leaves the live page stale or 404
  • Users asking to share or refresh a link need a live, openable URL from a real publish, not a dry-run preview

Add intent routing table and a dedicated publish-or-refresh share workflow
so agents share live URLs instead of stopping at dry-run previews.

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong samzong merged commit 3e0dd47 into main Jun 24, 2026
1 check passed
@samzong samzong deleted the docs/recall-share-routing branch June 24, 2026 08:13

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 987ab9292f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/recall/SKILL.md

| Intent | Example prompts | Workflow |
| --- | --- | --- |
| Share a live link | "分享会话", "分享这个对话", "share this session", "给我链接" | Publish Or Refresh Share Link |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Require explicit session context before publishing links

Because this row treats the bare prompt “给我链接” as a share intent, and the new share workflow says to execute immediately without a dry run, an ordinary Chinese request for a link can route into publishing the current transcript to Cloudflare Pages. Please require explicit session/share wording, such as “分享这个会话” or “给我这个会话的链接”, before invoking the publish workflow.

Useful? React with 👍 / 👎.

Comment thread skills/recall/SKILL.md
```bash
recall session list --project /absolute/project/path --source <source> --limit 1 --sort updated --sync --format json
```
- Named or older session: add `--query "<keywords>"` and/or `--time 7d`, still prefer `--sort updated`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Do not rely on updated sort with a query

When --query is added, cmd_session_list takes the hybrid-search branch and never applies sort, so --sort updated is accepted but ignored and results remain search-ordered. For a named share/update request with multiple matching sessions, this can select and publish the wrong session rather than the latest matching one; either avoid pairing --query with --sort updated here or update the CLI to support that ordering.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant