fix(search-service): fix global search issue

[Sourav-kashyap]

fix global search issue
GH-2553

Description

In Global Search, users are unable to see search results when searching for names containing . (dot), for example: Deal 10.1.
The issue originates from the _formatAndSanitize method in query.builder.ts, which is used by search.provider.ts.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• Performed a self-review of my own code
• npm test passes on your machine
• New tests added or existing tests modified to cover all changes

[Copilot]

Pull request overview

Fixes PostgreSQL global-search token sanitization so search terms containing dots (e.g., Deal 10.1) are preserved instead of being split/removed, and adds unit coverage for the updated sanitization behavior.

Changes:

• Update PsqlQueryBuilder._formatAndSanitize to allow . in tokens and strip unsafe to_tsquery operators.
• Add unit tests covering dotted terms (decimals, project names, IPs) and mixed punctuation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
services/search-service/src/classes/psql/query.builder.ts	Adjusts match sanitization/tokenization to preserve dots for PostgreSQL full-text search queries.
services/search-service/src/tests/unit/psql/query.builder.unit.ts	Adds targeted unit tests for _formatAndSanitize with dotted search terms and mixed characters.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[vaibhavbhalla2505]

I have checked it, and it's working fine now.

before:

now:

[sonarqubecloud]

SonarQube reviewer guide

Summary: Add comprehensive unit tests and improve the _formatAndSanitize method to preserve dots in search terms while properly handling PostgreSQL full-text search special characters.

Review Focus: The core logic change in _formatAndSanitize modifies character sanitization by preserving dots (essential for version numbers and project names) while explicitly removing PostgreSQL tsquery operators. Verify that the new regex patterns correctly handle edge cases and don't introduce security vulnerabilities or unexpected query behavior. Pay special attention to the trailing dot removal logic and the two-stage filtering approach.

Start review at: services/search-service/src/classes/psql/query.builder.ts. This file contains the functional change that alters search query behavior; the unit tests should be reviewed after confirming the implementation is correct.

💬 Please send your feedback

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
51.0% Duplication on New Code

See analysis details on SonarQube Cloud