feat(client): make Chat the default landing screen (was Notes) #373

Merged

james merged 1 commit from default-screen-chat into main 2026-06-29 14:59:34 +00:00

Conversation 1 Commits 1 Files changed 3 +8 -8

james commented 2026-06-29 14:49:03 +00:00

Owner

Copy link

The app landed on Notes on open and after sign-in. Make Chat the default — it's the primary surface now.

Updates the index redirect and every post-auth landing target, all /notes → /chat:

• app/index.tsx — <Redirect href> (+ its comment).
• app/login.tsx — post-login router.replace, the OAuth web next param, the OAuth native post-signin router.replace, and a stale comment.
• app/register.tsx — post-register router.replace.

The Notes screen and /notes route stay (still routable — it's intentionally out of the sidebar; only the default landing changes). A full rg "/notes" over the routing code comes back clean afterward (only /api/notes API-URL hits remain, which are unrelated).

Verification (local)

• typecheck ✓ · lint ✓ · test ✓ 172 (the two /notes test hits are an API URL and the "notes is not in the sidebar" nav assertion — both correctly unchanged) · semgrep ✓ 0 findings
• Only the 3 routing files changed.

Direct request (no ticket).

🤖 Generated with Claude Code

The app landed on **Notes** on open and after sign-in. Make **Chat** the default — it's the primary surface now. Updates the index redirect and every post-auth landing target, all `/notes` → `/chat`: - `app/index.tsx` — `<Redirect href>` (+ its comment). - `app/login.tsx` — post-login `router.replace`, the OAuth web `next` param, the OAuth native post-signin `router.replace`, and a stale comment. - `app/register.tsx` — post-register `router.replace`. The Notes screen and `/notes` route stay (still routable — it's intentionally out of the sidebar; only the default landing changes). A full `rg "/notes"` over the routing code comes back clean afterward (only `/api/notes` API-URL hits remain, which are unrelated). ## Verification (local) - `typecheck` ✓ · `lint` ✓ · `test` ✓ 172 (the two `/notes` test hits are an API URL and the "notes is not in the sidebar" nav assertion — both correctly unchanged) · semgrep ✓ 0 findings - Only the 3 routing files changed. Direct request (no ticket). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

james added 1 commit 2026-06-29 14:49:03 +00:00

feat(client): make Chat the default landing screen (was Notes) ... 11e2972015

The app landed on Notes on open and after sign-in. Make Chat the default,
since it's the primary surface now. Updates the index redirect and every
post-auth landing target — post-login + post-register router.replace, the
OAuth web `next` param, and the OAuth native post-signin replace — from
/notes to /chat (plus a stale comment). The Notes screen + route stay
(routable, just no longer the default).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

forgejo-actions commented 2026-06-29 14:57:27 +00:00

Copy link

📊 Test coverage

Patch coverage: no testable lines changed.

Overall (app/, lib/, db/, excluding UI per ADR-0019):

Metric	Value	Soft target
Lines	79.4% ✅	≥ 50%
Branches	71.2% ⚠️	≥ 75%
Functions	80.5%	informational

Soft thresholds per ADR-0019. Coverage is informational and does not block merge.

<!-- coverage-comment --> ## 📊 Test coverage **Patch coverage:** no testable lines changed. **Overall** (`app/`, `lib/`, `db/`, excluding UI per ADR-0019): | Metric | Value | Soft target | |---|---|---| | Lines | 79.4% ✅ | ≥ 50% | | Branches | 71.2% ⚠️ | ≥ 75% | | Functions | 80.5% | informational | Soft thresholds per [ADR-0019](docs/adr/0019-coverage-soft-targets.md). Coverage is informational and does not block merge.

james merged commit c77be030f8 into main 2026-06-29 14:59:34 +00:00

james deleted branch default-screen-chat 2026-06-29 14:59:35 +00:00

james referenced this pull request from a commit 2026-06-29 14:59:35 +00:00

Merge pull request 'feat(client): make Chat the default landing screen (was Notes)' (#373) from default-screen-chat into main

james referenced this pull request from a commit 2026-06-29 15:10:53 +00:00

fix(client): back navigation follows history, not always the landing screen

james referenced this pull request 2026-06-29 15:11:04 +00:00

fix(client): back navigation follows history, not always the landing screen #374

james referenced this pull request from a commit 2026-06-29 16:26:32 +00:00

test(e2e): shared session, db reset, and admin spec

james referenced this pull request 2026-06-29 16:27:03 +00:00

test(e2e): shared session, db reset, and admin spec #377

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

area:auth

Authentication and authorization

  

area:ci

Forgejo Actions, build, test, security, release

  

area:db

Database abstraction, migrations, persistence

  

area:infra

Containerization, deployment, foundational tooling

  

area:native

React Native app (Android + Linux Flatpak)

  

area:pwa

PWA / web frontend

  

area:service

Backend service, API routes, persistence

  

epic

High-level tracker; links several feature tickets

  

feature

A user-facing feature

  

foundation

Foundational work that other tickets build on

No labels

area:auth

area:ci

area:db

area:infra

area:native

area:pwa

area:service

epic

feature

foundation

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

james/carol!373

No description provided.