fix: navigate to config from the project page (#1424)

# Fix Config Button Navigation to Settings Sidebar

## Problem
Clicking the **Config** button in the project navigation bar resulted in
a 404 error instead of opening the project settings sidebar.

**Steps to Reproduce:**
1. Open the home page (Dashboard)
2. Click on any project to navigate to `/projects/{projectId}/explorer`
3. Click the **Config** button in the top navigation
4. **Expected:** Navigate to `/projects/{projectId}/explorer/settings`
and open settings sidebar
5. **Actual:** Navigate to `/projects/{projectId}/settings` → 404 error,
`projectId` becomes `undefined`

## Root Cause
The navigation issue occurred during the React Router redirect
transition. When navigating from the dashboard to a project:

1. The route initially lands at `/projects/{projectId}` (index route)
2. React Router then redirects to `/projects/{projectId}/explorer`
3. During this transition, if the Config button was clicked, the
`useNavigateWithSettings` hook would capture the base path as
`/projects/{projectId}` (without `/explorer`)
4. This resulted in navigating to `/projects/{projectId}/settings`
instead of `/projects/{projectId}/explorer/settings`
5. Since no route exists for `/projects/{projectId}/settings` directly,
the app returned a 404 error

## Solution
Modified `src/utilities/navigation.ts` in the `useNavigateWithSettings`
function to intelligently detect and handle project index routes:

- **Detection:** Added regex pattern to detect when the base path
matches `/projects/{projectId}` without a specific page
- **Default Behavior:** When navigating to settings from the project
index route (without `/explorer`, `/sessions`, or `/deployments`),
automatically append `/explorer/` before the settings path
- **Result:** Ensures navigation always produces
`/projects/{projectId}/explorer/settings` instead of
`/projects/{projectId}/settings`

## What type of PR is this? (check all applicable)

- [ ] 💡 (feat) - A new feature (non-breaking change which adds
functionality)
- [ ] 🔄 (refactor) - Code Refactoring - A code change that neither fixes
a bug nor adds a feature
- [x] 🐞 (fix) - Bug Fix (non-breaking change which fixes an issue)
- [ ] 🏎 (perf) - Optimization
- [ ] 📄 (docs) - Documentation - Documentation only changes
- [ ] 📄 (test) - Tests - Adding missing tests or correcting existing
tests
- [ ] ⚙️ (ci) - Continuous Integrations - Changes to our CI
configuration files and scripts (example scopes: Travis, Circle,
BrowserStack, SauceLabs)
- [ ] ☑️ (chore) - Chores - Other changes that don't modify src or test
files
- [ ] ↩️ (revert) - Reverts - Reverts a previous commit(s).

<!--
     For a timely review/response, please avoid force-pushing additional
     commits if your PR already received reviews or comments.
Before submitting a Pull Request, please ensure you've done the
following:
     - 👷‍♀️ Create small PRs. In most cases this will be possible.
     - ✅ Provide tests for your changes.
     - 📝 Use descriptive commit messages (as described below).
- 📗 Update any related documentation and include any relevant
screenshots.
     Commit Message Structure (all lower-case):
     <type>(optional ticket number): <description>
     [optional body]
-->

Author: RonenMars

e2e/project/triggers/triggerBase.spec.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 /* eslint-disable @typescript-eslint/naming-convention */
 import type { Page } from "@playwright/test";
 
@@ -91,9 +90,6 @@ async function modifyTrigger(
 		await expect(page.getByRole("heading", { name: "Configuration" })).toBeVisible();
 	}
 
-	const drawerVisible = await page.locator("#project-sidebar-config").isVisible();
-	console.log("Drawer visible before edit click:", drawerVisible);
-
 	const configureButtons = page.locator(`button[aria-label="Edit ${name}"]`);
 	await configureButtons.click();
 

e2e/project/webhookSessionTriggered.spec.ts

@@ -79,6 +79,7 @@ async function setupProjectAndTriggerSession({ dashboardPage, page, request }: S
 
 		await page.getByPlaceholder("Enter project name").fill(projectName);
 		await page.locator('button[aria-label="Create"]').click();
+		await page.waitForURL(/\/explorer\/settings/);
 		await expect(page.getByRole("heading", { name: "Configuration" })).toBeVisible();
 		// eslint-disable-next-line @typescript-eslint/no-unused-vars
 	} catch (error) {

src/autokitteh

@@ -37,7 +37,8 @@ export const useNavigateWithSettings = () => {
 	const location = useLocation();
 
 	return (to: string, options?: { replace?: boolean; state?: ProjectLocationState }) => {
-		const { basePath, settingsPath } = extractSettingsPath(location.pathname);
+		const currentPath = location.pathname;
+		const { basePath, settingsPath } = extractSettingsPath(currentPath);
 		const cleanBasePath = basePath.endsWith("/") ? basePath.slice(0, -1) : basePath;
 
 		let newBasePath: string;
@@ -50,7 +51,17 @@ export const useNavigateWithSettings = () => {
 			const projectBasePath = cleanBasePath.split("/").slice(0, -1).join("/");
 			newBasePath = `${projectBasePath}/${pageName}`;
 		} else {
-			newBasePath = `${cleanBasePath}/${to}`;
+			const projectPageMatch = cleanBasePath.match(/^\/projects\/[^/]+$/);
+			if (
+				projectPageMatch &&
+				!currentPath.includes("/explorer") &&
+				!currentPath.includes("/sessions") &&
+				!currentPath.includes("/deployments")
+			) {
+				newBasePath = `${cleanBasePath}/explorer/${to}`;
+			} else {
+				newBasePath = `${cleanBasePath}/${to}`;
+			}
 		}
 
 		let finalPath: string;