Skip to content

feat: show favicon in fetch output - #17

Closed
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sdk-cli-sync-fetch-favicon-20260821
Closed

cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sdk-cli-sync-fetch-favicon-20260821

Conversation

@cursor

@cursor cursor Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The public JS SDK fetch response and fetch task output require a favicon URL, but the CLI's human-readable formatter omitted it.
  • Show the favicon URL for synchronous fetches and completed fetch tasks.
  • Add unit and integration coverage and update the README.

Validation

  • npm run build
  • npm run lint
  • npm test (23 files, 179 tests passed)
Open in Web View Automation 

@Turanic
Turanic self-requested a review August 21, 2026 09:13
@Turanic

Turanic commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The response already support additional response fields with json output, so this change is not required per say. No need to add it to the "human readable output".

@Turanic Turanic closed this Aug 21, 2026
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