Skip to content

fix: Fix spelling errors + wrongly formatted json in prompts#1065

Open
northern-64bit wants to merge 7 commits intoItzCrazyKns:masterfrom
northern-64bit:patch-1
Open

fix: Fix spelling errors + wrongly formatted json in prompts#1065
northern-64bit wants to merge 7 commits intoItzCrazyKns:masterfrom
northern-64bit:patch-1

Conversation

@northern-64bit
Copy link

@northern-64bit northern-64bit commented Mar 16, 2026

Summary by cubic

Fixed spelling errors and an invalid JSON example across search prompts, agents, and the researcher web search action to improve clarity and prevent parsing errors.

  • Bug Fixes
    • Spelling/wording: “independant”→“independent” and “converastion”→“conversation” in src/lib/prompts/search/classifier.ts; “feature”→“features” in src/lib/prompts/search/researcher.ts; “souce”→“source” and “showed”→“shown” in both src/lib/agents/search/index.ts and src/lib/agents/search/api.ts; “this tools”→“this tool” in src/lib/agents/search/researcher/actions/webSearch.ts.
    • Fixed JSON example in src/lib/prompts/search/classifier.ts by removing a trailing comma.

Written for commit 138c56e. Summary will update on new commits.

@northern-64bit northern-64bit changed the title Fix spelling errors + wrongly formatted json in prompts fix: Fix spelling errors + wrongly formatted json in prompts Mar 16, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/lib/prompts/search/classifier.ts">

<violation number="1" location="src/lib/prompts/search/classifier.ts:59">
P2: The prompt claims to provide a JSON output format, but still contains invalid JSON type placeholders (`boolean`/`string`), making the formatting fix incomplete.</violation>
</file>

Since this is your first cubic review, here's how it works:

  • cubic automatically reviews your code and comments on bugs and improvements
  • Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
  • Add one-off context when rerunning by tagging @cubic-dev-ai with guidance or docs links (including llms.txt)
  • Ask questions if you need clarification on any suggestion

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Eternalhazed added a commit to Eternalhazed/perplexica that referenced this pull request Mar 18, 2026
…CrazyKns#1065)

Fixes "independant" -> "independent", "converastion" -> "conversation",
and removes trailing comma in JSON example that could cause parse failures.

From upstream PR ItzCrazyKns#1065 (northern-64bit)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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