Skip to content

Raycast: address Store review feedback - #16

Merged
fatwang2 merged 1 commit into
mainfrom
claude/project-thread-k88liz
Nov 3, 2026
Merged

fatwang2 merged 1 commit into
mainfrom
claude/project-thread-k88liz

Conversation

@fatwang2

@fatwang2 fatwang2 commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Fixes the automated review findings on the Raycast Store submission (raycast/extensions#33095).

  • Move contributor notes (raycast/AGENTS.md, README Development and Releasing sections) to docs/raycast.md, so they no longer ship with the extension
  • Drop the redundant "subtitle": "Pulse"; declare "platforms": ["macOS"]
  • Catalog writes use a unique temp file (pid + UUID)
  • A typed ^… ticker (e.g. ^FTSE) counts as an index, so the Indices filter keeps it
  • Symbol-list failure toast gets a Retry action
  • Show/Hide Chart (⌘D) moves to its own section, so it is no longer the ⌘↵ secondary action

Checks: npm test (16 pass), npm run lint, npm run build.

After merge, rerun Release Pulse for Raycast to update the submission.


Generated by Claude Code

- Move contributor notes (AGENTS.md, Development, Releasing) out of raycast/ to docs/raycast.md
- Drop the redundant command subtitle; declare macOS as the platform
- Unique temp file per catalog write
- A typed ^ ticker is an index, so the Indices filter keeps it
- Retry action on the symbol-list failure toast; Show/Hide Chart in its own section

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DuYf7vdDtMHbY6YTKJrEqJ
@fatwang2
fatwang2 merged commit 1b1004d into main Nov 3, 2026
4 checks passed
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.

2 participants