Skip to content

fix(web): include non-pico channel sessions in history API#2021

Closed
Alix-007 wants to merge 2 commits intosipeed:mainfrom
Alix-007:fix/issue-1996-session-history-all-channels
Closed

fix(web): include non-pico channel sessions in history API#2021
Alix-007 wants to merge 2 commits intosipeed:mainfrom
Alix-007:fix/issue-1996-session-history-all-channels

Conversation

@Alix-007
Copy link
Copy Markdown
Contributor

@Alix-007 Alix-007 commented Mar 25, 2026

Summary

  • include non-pico session keys in GET /api/sessions instead of filtering to pico-only keys
  • keep backward compatibility for existing pico short IDs while allowing full non-pico keys for GET /api/sessions/{id} and DELETE /api/sessions/{id}
  • add regression tests for non-pico JSONL and legacy JSON session history flows

Fixes #1996.

Verification

  • Command: go test ./web/backend/api
  • Result: ok github.com/sipeed/picoclaw/web/backend/api (cached)

@sipeed-bot sipeed-bot bot added type: bug Something isn't working domain: channel go Pull requests that update go code labels Mar 25, 2026
@KristjanKruusRIA
Copy link
Copy Markdown

Duplicate of #2002

@Alix-007
Copy link
Copy Markdown
Contributor Author

Closing this as a duplicate of #2002 to avoid splitting review effort across two PRs for the same issue. If #2002 stalls or its scope changes materially, I can reopen the work on a fresh clean base.

@Alix-007 Alix-007 closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: channel go Pull requests that update go code type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web UI: Chat history only shows pico (web) sessions, hides all other channels

2 participants