Skip to content

feat(skills): surface Company Brain-created skills - #1412

Merged
graphite-app[bot] merged 1 commit into
mainfrom
sreedharsreeram/buffalo
Aug 5, 2026
Merged

feat(skills): surface Company Brain-created skills#1412
graphite-app[bot] merged 1 commit into
mainfrom
sreedharsreeram/buffalo

Conversation

@sreedharsreeram

@sreedharsreeram sreedharsreeram commented Aug 4, 2026

Copy link
Copy Markdown
Member

What and why

Make skills created through Company Brain visible in the existing Settings catalog without a manual reload. The UI labels Slack-originated skills and refreshes the catalog while the Settings page is open.

flowchart LR
  A[Slack approval] --> B[Company Brain skill catalog]
  B --> C[Skills API query]
  C --> D[Settings Skills list]
  D --> E[Created by Company Brain label]
Loading

Validation

  • bunx biome check apps/web/components/settings/company-brain-skills.tsx apps/web/components/settings/company-brain-skills/domain.ts apps/web/components/settings/company-brain-skills/skill-row.tsx apps/web/hooks/use-brain-skills.impl.ts — passed

Impact

Settings polling refreshes every 15 seconds and on window focus. It does not change the Nova/browser chat workflow or create skills from that surface.

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude

claude Bot commented Aug 4, 2026

Copy link
Copy Markdown

Claude finished @sreedharsreeram's task in 1m 31s —— View job


Review Complete

  • Fetch PR diff and analyze changes
  • Check for bugs, security issues, and logic errors
  • Review surrounding code for context
  • Submit review summary

Result: No issues found. This PR cleanly adds a badge to identify Company Brain-created skills and enables automatic refresh (15s polling) so approved Slack drafts appear without manual reload. The code handles edge cases gracefully.

Score: 10/10 — Ready to ship.

@claude claude 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.

Overview: Surfaces Company Brain-created skills with a "Created by Company Brain" badge and adds automatic refresh (15s polling + refetch on window focus) to show newly approved Slack drafts without manual reload.

Issues found: None — this looks good to ship.

The code is well-structured:

  • SkillOrigin type cleanly extends NewSkillOrigin with the "slack" variant
  • skillOriginLabel() gracefully returns null for non-Slack origins, so OriginBadge safely renders nothing
  • The polling configuration (staleTime: 0, refetchOnWindowFocus: true, refetchInterval: 15_000) is appropriate for real-time visibility of approved skills

Score: 10/10

@sohamd22 sohamd22 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Read through the code, LGTM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 570ed22 Commit Preview URL

Branch Preview URL
Aug 05 2026, 03:45 AM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 570ed22 Aug 05 2026, 03:43 AM

@graphite-app

graphite-app Bot commented Aug 5, 2026

Copy link
Copy Markdown

Merge activity

## What and why

Make skills created through Company Brain visible in the existing Settings catalog without a manual reload. The UI labels Slack-originated skills and refreshes the catalog while the Settings page is open.

```mermaid
flowchart LR
  A[Slack approval] --> B[Company Brain skill catalog]
  B --> C[Skills API query]
  C --> D[Settings Skills list]
  D --> E[Created by Company Brain label]
```

## Validation

- `bunx biome check apps/web/components/settings/company-brain-skills.tsx apps/web/components/settings/company-brain-skills/domain.ts apps/web/components/settings/company-brain-skills/skill-row.tsx apps/web/hooks/use-brain-skills.impl.ts` — passed

## Impact

Settings polling refreshes every 15 seconds and on window focus. It does not change the Nova/browser chat workflow or create skills from that surface.
@graphite-app
graphite-app Bot force-pushed the sreedharsreeram/buffalo branch from 94a3b05 to 570ed22 Compare August 5, 2026 03:42
@graphite-app
graphite-app Bot merged commit 570ed22 into main Aug 5, 2026
6 of 8 checks passed
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