docs(plans): close out chat-links-full-navigation and slack-header-link

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TGqwPjt3v4eMfiHpPmCKnR

Author: themightychris

plans/chat-links-full-navigation.md

@@ -1,9 +1,10 @@
 ---
-status: in-progress
+status: done
 depends: [chat-redirect]
 specs:
   - specs/screens/chat.md
 issues: [230]
+pr: 233
 ---
 
 # Plan: In-app links to /chat navigate to the server
@@ -36,10 +37,10 @@ a lint rule rejects router links whose literal target starts with `/chat`,
 
 ## Validation
 
-- [ ] `npm run type-check && npm run lint && npm test` clean.
-- [ ] The rule flags the five links before the fix (checked by running
+- [x] `npm run type-check && npm run lint && npm test` clean.
+- [x] The rule flags the five links before the fix (checked by running
   lint against the unfixed tree).
-- [ ] In the built app, clicking footer "Slack" leaves the SPA for the
+- [x] In the built app, clicking footer "Slack" leaves the SPA for the
   Slack SSO start URL instead of rendering the 404 page.
 
 ## Risks / unknowns
@@ -49,4 +50,14 @@ a lint rule rejects router links whose literal target starts with `/chat`,
 
 ## Notes
 
+- Five links, not the three the issue named: the project page's "Chat
+  Channel" button and the project card's "Chat" had the same bug.
+- Verified on the devbox build of develop at 0911423: clicking the footer
+  "Slack" left the SPA through Slack's SSO start to the production IdP's
+  sign-in (Slack's IdP is codeforphilly.org) instead of the 404 page.
+- The Vite dev server doesn't proxy `/chat`, so this can't be checked
+  under `npm run dev`; it needs the built app.
+
 ## Follow-ups
+
+- **None.**

plans/slack-header-link.md

@@ -1,9 +1,10 @@
 ---
-status: in-progress
+status: done
 depends: [chat-links-full-navigation]
 specs:
   - specs/behaviors/app-shell.md
 issues: [231]
+pr: 235
 ---
 
 # Plan: Slack in the header
@@ -43,7 +44,7 @@ content links); changing `/chat` itself.
 - [x] At 1024px and 1280px the desktop header keeps every content link on
   one line with Slack and GitHub showing; at 768px the mobile header shows.
 - [x] At 390px the sheet lists Slack before GitHub.
-- [ ] Clicking the header icon opens the Slack SSO start in a new tab.
+- [x] Clicking the header icon opens the Slack SSO start in a new tab.
 
 ## Risks / unknowns
 
@@ -52,4 +53,15 @@ content links); changing `/chat` itself.
 
 ## Notes
 
+- The 768px overflow predates this change: develop already wrapped "Help
+  Wanted" and pushed Volunteer off the edge there, most likely since
+  Events joined the content cluster. Moving the breakpoint fixed that too.
+- The header icon is the same `/chat` anchor as the footer link verified
+  in [chat-links-full-navigation](chat-links-full-navigation.md), plus
+  `target="_blank"`; checked by markup rather than a second click-through.
+- Chose an icon beside GitHub over a "Chat" text item in the content
+  cluster.
+
 ## Follow-ups
+
+- **None.**