Skip to content

fix(developer): omit duplicate URLs and empty separators - #386

Merged
PhantomInTheWire merged 1 commit into
mainfrom
fix/developer-url-dedup
Aug 26, 2026
Merged

fix(developer): omit duplicate URLs and empty separators#386
PhantomInTheWire merged 1 commit into
mainfrom
fix/developer-url-dedup

Conversation

@PhantomInTheWire

@PhantomInTheWire PhantomInTheWire commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • omit the bare URL when the result ID already contains that exact URL
  • skip separators for empty or whitespace-only passage blocks
  • keep server-shaped passage text uncapped

On the captured how to use io uring in rust response, the current-source formatter drops from 3,580 to 3,413 o200k_base tokens, a 4.7% reduction.

Verification

  • pnpm run lint
  • pnpm run test (74 passed)

Summary by cubic

Developer search output no longer prints a duplicate bare URL when it鈥檚 already in the result ID and no longer inserts '---' between empty passages. This trims noise and reduces token usage while keeping server-shaped passages intact.

Bug Fixes

  • Skip printing the bare URL when it already appears in the result ID; keep it when different (e.g., base vs child).
  • Drop empty or whitespace-only passages to avoid stray '---' separators.
  • Preserve server-shaped passages in full (no client-side capping).
  • On the captured "how to use io uring in rust" response, output drops from 3,580 to 3,413 o200k_base tokens (~4.7%).

Written for commit d3cea15. Summary will update on new commits.

Review in cubic

@PhantomInTheWire

Copy link
Copy Markdown
Member Author

@cubic-dev-ai review

@cubic-dev-ai

cubic-dev-ai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review

@PhantomInTheWire I have started the AI code review. It will take a few minutes to complete.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread src/developer.ts Outdated
@PhantomInTheWire
PhantomInTheWire force-pushed the fix/developer-url-dedup branch from 59fa032 to d3cea15 Compare August 25, 2026 11:57
@PhantomInTheWire

Copy link
Copy Markdown
Member Author

@cubic-dev-ai review

Please re-review exact head d3cea15addff80e9fa68098d03a652e1ada5b418. The prior exact-URL finding has a fix and regression. Lint, build, and 74 tests pass.

@cubic-dev-ai

cubic-dev-ai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review

Please re-review exact head d3cea15addff80e9fa68098d03a652e1ada5b418. The prior exact-URL finding has a fix and regression. Lint, build, and 74 tests pass.

@PhantomInTheWire I have started the AI code review. It will take a few minutes to complete.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Auto-approved: Focused bug fix: drops duplicate bare URLs and empty/whitespace-only passage separators in developer search output, with tests pinning the new behavior.

Re-trigger cubic

@PhantomInTheWire
PhantomInTheWire merged commit 5365a38 into main Aug 26, 2026
2 checks passed
PhantomInTheWire added a commit that referenced this pull request Aug 26, 2026
Publish the renderer deduplication fix from #386.
@PhantomInTheWire
PhantomInTheWire deleted the fix/developer-url-dedup branch August 27, 2026 15:16
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