feat: show project Go to Settings for credential queries #2526

HarshMN2345 · 2025-10-29T08:16:27Z

What does this PR do?

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

New Features
- Project search now recognizes credential-related queries (endpoint, API key, project ID) and provides direct navigation to Settings
- Enhanced project search filtering to include project ID, region, and endpoint in search results
Improvements
- Better text parameter handling for region endpoint display

railway-app · 2025-10-29T08:16:31Z

This PR was not deployed automatically as @HarshMN2345 does not have access to the Railway project.

In order to get automatic PR deploys, please add @HarshMN2345 to your workspace on Railway.

appwrite · 2025-10-29T08:16:33Z

Console

Project ID: 688b7bf400350cbd60e9

Sites (1)

Site	Status	Logs	Preview	QR
console-stage `688b7cf6003b1842c9dc`	Ready	View Logs	Preview URL

Tip

Appwrite has a Discord community with over 16 000 members.

coderabbitai · 2025-10-29T08:16:43Z

Walkthrough

This pull request enhances the project search functionality and improves text truncation parameter handling. The projects searcher now detects credential-related query terms to short-circuit to Settings navigation, retrieves API endpoints for richer searchable context, and implements tokenized filtering across project name, ID, region, and endpoint fields. Simultaneously, the truncateText action is extended to accept optional text parameters, allowing callers to provide initial text and updates without relying solely on node.textContent, with regionEndpoint updated to pass region names to the action.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Areas requiring extra attention:

Credential query detection logic in projects.ts: Verify the term matching and Settings navigation behavior handle edge cases correctly and do not unintentionally trigger on partial matches
Tokenized filtering implementation: Confirm the token parsing and "all tokens present" requirement work as expected for various query patterns and special characters
API endpoint retrieval and project store integration: Ensure getApiEndpoint calls are properly error-handled and the project store context is available in the searcher context
truncateText parameter propagation: Verify that the optional text parameter is correctly passed from regionEndpoint and that the update method properly handles undefined newText fallback to node.textContent
Backward compatibility: Check that existing callers of truncateText without the new text parameter continue to function correctly

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "feat: show project Go to Settings for credential queries" directly describes the primary functionality added in the main file being modified (src/lib/commandCenter/searchers/projects.ts). The changes implement detection of credential-like query terms (endpoint, api key, project id) and provide a "Go to Settings" navigation shortcut, which is exactly what the title conveys. The supporting changes to the truncateText action and regionEndpoint component are enabling modifications that don't alter the core purpose. The title is concise, specific, and clear enough that a teammate scanning the git history would understand the feature being added.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat-SER-443-make-api-endpoint-searchable-command-center

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (1)

src/lib/commandCenter/searchers/projects.ts (1)

56-70: Consider using map instead of flatMap for clarity.

Since each project produces exactly one navigation item, using map instead of flatMap would be more direct and slightly more efficient.

-        .flatMap((project) => {
+        .map((project) => {
             const href = `${base}/project-${project.region}-${project.$id}`;
 
             const label = project.name;
 
-            return [
-                {
-                    label,
-                    callback: () => {
-                        goto(href);
-                    },
-                    group: 'projects'
-                }
-            ];
+            return {
+                label,
+                callback: () => {
+                    goto(href);
+                },
+                group: 'projects'
+            };
         });

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8b9feaf and 2003299.

📒 Files selected for processing (3)

src/lib/commandCenter/searchers/projects.ts (1 hunks)
src/lib/components/id.svelte (2 hunks)
src/lib/components/regionEndpoint.svelte (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

src/lib/commandCenter/searchers/projects.ts (3)

src/routes/(console)/project-[region]-[project]/store.ts (1)

project (11-11)

src/lib/stores/sdk.ts (2)

sdk (153-176)

getApiEndpoint (47-59)

src/lib/stores/organization.ts (1)

organization (62-62)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: build
GitHub Check: e2e

🔇 Additional comments (2)

src/lib/components/regionEndpoint.svelte (1)

41-41: LGTM! Correctly leverages updated truncateText signature.

Passing region?.name explicitly to the action enables proper reactive truncation when the region changes, and the optional chaining safely handles undefined cases.

src/lib/components/id.svelte (1)

32-33: LGTM! Backward-compatible enhancement for explicit text management.

The optional parameters enable callers to explicitly provide text for truncation while maintaining backward compatibility with existing usage that relies on node.textContent. The update method correctly receives new text values from Svelte's action lifecycle when parameters change.

Also applies to: 73-74

src/lib/commandCenter/searchers/projects.ts

TorstenDittmann · 2025-10-30T10:26:53Z

src/lib/components/id.svelte

+    export function truncateText(node: HTMLElement, text?: string) {
+        let originalText = text ?? node.textContent;


why is this needed? 🤔

actions should always work with elements and usually shouldnt have a new context passed 👍🏻

TorstenDittmann · 2025-10-30T10:28:57Z

src/lib/commandCenter/searchers/projects.ts

+    const q = query.toLowerCase().trim();
+    const wantsCredentials =
+        q.includes('endpoint') ||
+        q.includes('api key') ||
+        q.includes('api-key') ||
+        q.includes('apikey') ||
+        q.includes('project id') ||
+        q.includes('project-id') ||
+        q === 'id' ||
+        q.endsWith(' id') ||
+        q.startsWith('end') ||
+        q.includes('api end') ||
+        (q.includes('api') && q.includes('end'));
+


I think we can make this simpler:

const keywords = [ 'endpoint', 'api key', 'api-key', 'apikey', 'project id', 'project-id', 'api end' ]; const wantsCredentials = keywords.some(k => q.includes(k));

something like this maybe

TorstenDittmann · 2025-10-30T10:29:51Z

src/lib/commandCenter/searchers/projects.ts

-            } as const;
+        .filter((project) => {
+            const endpoint = getApiEndpoint(project.region);
+            const searchable = [project.name, project.$id, project.region, endpoint]


i dont think we need the endpoint here 👍🏻

TorstenDittmann · 2025-10-30T10:32:15Z

src/lib/commandCenter/searchers/projects.ts

+        .flatMap((project) => {
+            const href = `${base}/project-${project.region}-${project.$id}`;
+
+            const label = project.name;
+
+            return [
+                {
+                    label,
+                    callback: () => {
+                        goto(href);
+                    },
+                    group: 'projects'
+                }
+            ];
        });


Suggested change

.flatMap((project) => {

const href = `${base}/project-${project.region}-${project.$id}`;

const label = project.name;

return [

{

label,

callback: () => {

goto(href);

},

group: 'projects'

}

];

});

.map((project) => {

const href = `${base}/project-${project.region}-${project.$id}`;

const label = project.name;

return {

label,

callback: () => {

goto(href);

},

group: 'projects'

};

});

why flatmap over map?

show only project Go to Settings for credential queries

2003299

coderabbitai bot reviewed Oct 29, 2025

View reviewed changes

src/lib/commandCenter/searchers/projects.ts Show resolved Hide resolved

HarshMN2345 requested a review from ItzNotABug October 29, 2025 08:21

TorstenDittmann reviewed Oct 30, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: show project Go to Settings for credential queries #2526

feat: show project Go to Settings for credential queries #2526

HarshMN2345 commented Oct 29, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

railway-app bot commented Oct 29, 2025

Uh oh!

appwrite bot commented Oct 29, 2025 •

edited

Loading

Uh oh!

coderabbitai bot commented Oct 29, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

TorstenDittmann Oct 30, 2025

Uh oh!

TorstenDittmann Oct 30, 2025

Uh oh!

TorstenDittmann Oct 30, 2025

Uh oh!

TorstenDittmann Oct 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		export function truncateText(node: HTMLElement, text?: string) {
		let originalText = text ?? node.textContent;

feat: show project Go to Settings for credential queries #2526

Are you sure you want to change the base?

feat: show project Go to Settings for credential queries #2526

Conversation

HarshMN2345 commented Oct 29, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Test Plan

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Summary by CodeRabbit

Uh oh!

railway-app bot commented Oct 29, 2025

Uh oh!

appwrite bot commented Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Console

Uh oh!

coderabbitai bot commented Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Estimated code review effort

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

TorstenDittmann Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

TorstenDittmann Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

TorstenDittmann Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

TorstenDittmann Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

HarshMN2345 commented Oct 29, 2025 •

edited by coderabbitai bot

Loading

appwrite bot commented Oct 29, 2025 •

edited

Loading

coderabbitai bot commented Oct 29, 2025 •

edited

Loading