fix: publish app home view, wire worker focus check, store oauth token, init metrics, fix go version#7
Conversation
…blish Adds PublishView to SlackAPI interface and implements it in EventHandler. Controller now builds a full Block Kit Home tab showing neurotype, feature status, and command reference when user opens App Home.
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (19)
📝 WalkthroughWalkthroughThis PR adds optional Google Calendar integration, Slack App Home publishing, OAuth bot-user persistence, worker state checks, Prometheus startup initialization, Go 1.26 build updates, and minor frontend and repository cleanup. ChangesGoogle Calendar integration
Slack integration updates
Runtime and tooling updates
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant MCPToolHandler
participant GoogleCalendarClient
participant GoogleCalendarAPI
MCPToolHandler->>GoogleCalendarClient: CreateEvent or GetCurrentEvent with request context
GoogleCalendarClient->>GoogleCalendarAPI: Insert or list calendar events
GoogleCalendarAPI-->>GoogleCalendarClient: Event result
GoogleCalendarClient-->>MCPToolHandler: Event ID or current summary
sequenceDiagram
participant SlackEventHandler
participant FeatureController
participant SlackAPI
SlackEventHandler->>FeatureController: HandleAppHomeOpened
FeatureController->>FeatureController: Build preference and feature status blocks
FeatureController->>SlackAPI: PublishView with user ID and blocks
SlackAPI-->>FeatureController: Publish result
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
What
Fixes all remaining broken items from the audit: App Home view now publishes real preferences, worker focus:check uses Redis for velocity monitoring, OAuth creates a team record in DB, Prometheus metrics are initialized, and Go version is synced across CI and Dockerfiles.
Why
These were the actual remaining gaps. Users saw a blank App Home tab, the worker focus:check was a stub, OAuth tokens were discarded without creating any DB record, and Metrics/MCP had Go version mismatches.
Changes
views.publishAPIfocus:checknow reads Redis velocity counters and checks offered flags instead of logging a stub messageNewMetrics()called in main.go so Prometheus counters appear on /metrics endpointHow to Test
go build ./cmd/api && go build ./cmd/worker && go build ./cmd/mcp-server— all compilego test -race ./...— 35 tests passgo vet ./...— cleannpx tsc --noEmit && npm run lint— typecheck cleanChecklist
make lintpasses (go vet clean)make testpasses (35 tests, race clean)Summary by CodeRabbit
New Features
Improvements
Documentation