fix(developer): deduplicate URLs and clean separators

[PhantomInTheWire]

Summary

• omit the bare result URL when the result ID already contains that exact URL
• skip separators for empty passage blocks
• leave readable passages intact

Developer search already budgets passages server-side at 4,096 tokens in firecrawl/search#843. The CLI does not add a second cap, matching firecrawl/firecrawl-mcp-server#381.

Verification

• pnpm run format:check
• pnpm run type-check
• pnpm run build
• pnpm run test (433 passed)

The repository uses separate package-only release PRs. This change does not bump or publish version 1.23.1.

[PhantomInTheWire]

@cubic-dev-ai review

[cubic-dev-ai]

@cubic-dev-ai review

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

[cubic-dev-ai]

All reported issues were addressed across 2 files

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[PhantomInTheWire]

@cubic-dev-ai review

Please re-review exact head a6145c08440ba0bdbe4c68f80787147b01dc4c39. All three prior findings have fixes and regressions. Format, type-check, build, and 433 tests pass.

[cubic-dev-ai]

@cubic-dev-ai review

Please re-review exact head a6145c08440ba0bdbe4c68f80787147b01dc4c39. All three prior findings have fixes and regressions. Format, type-check, build, and 433 tests pass.

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

[cubic-dev-ai]

No issues found across 2 files

_{Re-trigger cubic}