GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||
GeezerVison is showing github.com — feat: add chat project summaries and pin telemetry by jaaydenh · Pull Request #32026 · coder/coder Conversation
Contributor
Adds the project summaries endpoint and pin telemetry on top of #32025. The Projects page grid and the sidebar's pinned projects both read these summaries.
The existing Verification
Dogfood UAT with a premium license checked the summary fields against seeded chats, member counts with user and group overlap, a user removed from the org and an Everyone share, and that the summaries' project IDs equal the list's in dev, sharing-disabled, re-enabled and release-build experiment-off modes. Example after the upgrade migration: {"name":"Alpha","pinned":true,"pin_order":1,"own_chat_count":0,"own_unread_count":0,"own_running":false,"own_needs_attention":false,"last_activity_at":null,"member_count":1}
{"name":"Bshared","pinned":false,"pin_order":0,"own_chat_count":0,"own_unread_count":0,"own_running":false,"own_needs_attention":false,"last_activity_at":null,"member_count":2}Note Most of this PR was written by AI agents (Xum, with UAT run through Coder Agents). The verification above is from those runs.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Add GET /api/experimental/chats/projects/summaries. It returns the projects from the project list with the caller's pin, numbered over the pins the caller can still see, and aggregates over the caller's own top-level, non-archived chats: chat and unread counts, running and needs-attention flags, latest activity, and how many users can use the project. Report ChatProjectPins in telemetry: pin and tombstone counts by source, and how many users pinned what share of their projects. Tests that check pin positions now read them through the summaries, so they also cover pins hidden after access is revoked.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 9ca6f65f7c
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Suspended and dormant users no longer inflate the pin telemetry, and project grants count only when chat sharing is enabled, matching the project summaries.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 4854a18c10
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
A group from another organization grants access only to its members who also belong to the project's organization, so member counts and pin telemetry now apply the same restriction.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: f054060774
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
At least 1 approving review is required to merge this pull request.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.