Skip to content

Circles tab lists topic Circles only; retire the Space Circle readers (LIVE-2078) - #4578

Merged
hellofrequencylab merged 26 commits into
mainfrom
claude/project-thread-a4t5ph
Nov 3, 2026
Merged

hellofrequencylab merged 26 commits into
mainfrom
claude/project-thread-a4t5ph

Conversation

@hellofrequencylab

Copy link
Copy Markdown
Owner

What

W3 of the Community wall (ADR-1938). A Space's conversation is its Community wall now, so:

  • The Space Circles tab lists topic Circles only and mounts no feed. A legacy Space Circle is filtered out until W5 archives it.
  • /spaces/<slug>/discussion forwards to /community (307, so no cached 308 if it moves again). The slug stays reserved.
  • The owner Circles console drops the Space Circle card and links Wall settings.
  • lib/spaces/space-discussion.ts, components/spaces/space-circle-card.tsx and the circle-space-info widget are deleted, and removed from the widget registry, module catalog, default /circles/* layout and admin-client baseline. circle-feed is parked in the reachability guard (the Frequency Circle shell draws FqCircleFeed; the Discussion band was its last mount).

Why

Daniel's ruling 2026-11-03: Space Circles become a Community wall. W2 shipped in #4550.

How to review (owner)

Needs the owner's phrase for these pin rewrites before merge:

  1. Probe LIVE-382 no longer requires the Space Circle On/Off card.
  2. Probe LIVE-384 no longer reads the Space Circle info board.
  3. Probe LIVE-421 now checks the Community wall feed and Community menu row instead of the Space Circle hub reader.
  4. Probe LIVE-519 drops its Space Circles page arm (the circle shell checks stay).
  5. Probe LIVE-523 now checks that /discussion forwards to /community and the Circles tab mounts no feed.
  6. lib/page-settings/default-layouts.test.ts rail order drops circle-space-info.

Ledger

LIVE-2078 closed (probe space-community-wall.mjs w3). The remaining is_space_primary readers already exclude or special-case the Space Circle and stop seeing it when W5 (LIVE-2080) archives it; noted on that row.

Verification

  • tsc --noEmit clean
  • eslint clean on touched files
  • full vitest: 2438 files, 27590 tests pass
  • check:backlog --probes=all: 2221 probes agree; check:admin-client, check:id-collisions pass

Generated by Claude Code

claude and others added 24 commits November 3, 2026 15:17
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
… NAMING amended

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…types, pgTAP proof

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
… uses latest posts read policy, fixes pgTAP fixture

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…ing (LIVE-2077 slice 1)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
… row with slice-1 note

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…unregistered visual mask

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…unity (LIVE-2077)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…77, split polish to LIVE-2082

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

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…t, LIVE-082, LIVE-421)

Daniel approved 2026-11-03 17:46Z (ADR-1938).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV18UVmNapUByFnwyS
…ebrow role, non-row file name)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012BMmZV5ygXEb5JWTKhgrn3
…cle readers

A Space's conversation is its Community wall now (ADR-1938). The Circles tab
lists topic Circles and mounts no feed, /spaces/<slug>/discussion forwards to
/community, and the owner Circles console links Wall settings in place of the
Space Circle card. space-discussion.ts, space-circle-card.tsx and the
circle-space-info widget are deleted, and leave the registry, module catalog,
default layout and admin-client baseline.

Probes LIVE-382, LIVE-384, LIVE-421, LIVE-519 and LIVE-523 and the circle rail
order test are rewritten to measure the wall that replaced those readers.

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

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
frequency-web Ready Ready Preview Nov 3, 2026 11:16pm UTC

Request Review

@hellofrequencylab
hellofrequencylab marked this pull request as ready for review November 3, 2026 20:38

Copy link
Copy Markdown
Owner Author

migration-readiness failed closed on 12d1b02 with TypeError ... stage=github-api from the runner. This PR adds no migration, so the check has nothing of its own to judge here; the same github-api failure is what #4574 is diagnosing. I re-ran the runner job once.


Generated by Claude Code

@hellofrequencylab
hellofrequencylab merged commit 82785bd into main Nov 3, 2026
18 checks passed

This branch was successfully deployed

1 active deployment
Preview — ed3b7c6b Deployed Nov 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants