Skip to content

feat(feed): notify browsers of canonical inventory changes - #21

Merged
autohub-codex[bot] merged 3 commits into
mainfrom
feat/command-05-feed-notifications
Nov 8, 2026
Merged

autohub-codex[bot] merged 3 commits into
mainfrom
feat/command-05-feed-notifications

Conversation

@autohub-codex

@autohub-codex autohub-codex Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Connect canonical Hub change subscriptions to feed refreshes and expose bounded same-origin change notifications. Add the browser notification client and coalesced latest-refresh helper. Part6 connects that client to the town renderer.

Part 5 of the approved eight-PR CottageCode command-center stack. Base: feat/command-04-command-views. This PR contains only this part's changes; consumers are staged in later parts where stated.

Review Contract

  • Intended outcome: Browser feed notifications.
  • Invariants and acceptance: One canonical change can refresh consumers without the history polling delay; bursts coalesce and later changes are not lost during an inflight refresh. SSE listeners, reconnects and shutdown are bounded; live endpoints stay disabled for the public sample.
  • Non-goals: Inventing task lifecycle in notifications or wiring the town consumer before part6.
  • Existing repository invariants in AGENTS.md remain in force.

Validation

  • Cumulative stage validation: npm test passed 500 tests, zero failures, on Node24.15.0.
  • Split from approved checkpoint 939a9fee14e15b6fc7dca37906c79343d77b836b. Subsequent review corrections are recorded in the stack; the preview remains on the original checkpoint branch.
  • Local Codex review completed; no unresolved actionable finding remains in this part. Baseline hosted review is complete; the later head completed its 10-minute automatic-review grace without another review. Smoke CI passed on the verified head. Staged consumer integration is documented above.
  • Size: 283 changed lines. No breaking feed migration is intended; additive fields retain unavailable/partial evidence semantics.

Implementation harness: Codex desktop. GitHub identity: autohub-codex[bot].

Stack navigation

Review and land in order; each PR targets the previous part. Readiness applies to that declared base and must be rechecked when bases change.

Part PR Base
1 Shared evidence and safe replies main
2 Hub telemetry and inventory stream feat/command-01-evidence
3 Village life and explicit family paths feat/command-02-telemetry
4 Command views and workbench feat/command-03-village-life
5 Browser feed notifications feat/command-04-command-views
6 Town and inspector integration feat/command-05-feed-notifications
7 Exploration harness feat/command-06-town-integration
8 Checkpoint documentation feat/command-07-exploration

@jack-arturo

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-08T07:48:02.201801Z 303c262 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 303c26286e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/feed.mjs Resolved
@autohub-codex autohub-codex Bot added babysit:active Babysit agent is actively working this PR and removed babysit:waiting-codex 👀 Codex reviewing current head labels Nov 8, 2026
@autohub-codex autohub-codex Bot added babysit:waiting-codex 👀 Codex reviewing current head babysit:ready 👍 Codex all clear + gates pass; human may merge and removed babysit:active Babysit agent is actively working this PR babysit:waiting-codex 👀 Codex reviewing current head labels Nov 8, 2026
@autohub-codex

autohub-codex Bot commented Nov 8, 2026

Copy link
Copy Markdown
Contributor Author

Babysit complete — verified green at df216bdcd9af2aae165c75dbbe5e6274ddb0331e.

Part 5/8. Ready against feat/command-04-command-views; land the stack in order and recheck after changing a base.
Local Codex review: no unresolved actionable finding. Cumulative npm test: 500 passing. Codex completed the baseline review; its 10-minute smart-review grace expired on this later head without a new review. No additional review request was posted. No unresolved active Codex threads remain.
Hosted smoke CI passed with its syntax and feed-boot steps executed. Mergeability: MERGEABLE. Size: 283 changed lines.

No merge performed. Harness: Codex desktop via autohub-codex[bot].

@autohub-codex
autohub-codex Bot changed the base branch from feat/command-04-command-views to main November 8, 2026 10:10
@autohub-codex
autohub-codex Bot merged commit f35dba7 into main Nov 8, 2026
2 checks passed
@autohub-codex autohub-codex Bot removed the babysit:ready 👍 Codex all clear + gates pass; human may merge label Nov 8, 2026
@autohub-codex
autohub-codex Bot deleted the feat/command-05-feed-notifications branch November 8, 2026 20:44
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.

1 participant