Skip to content

Commit 3de8197

Browse files
siracusa5claude
andcommitted
fix(review): address code and doc review findings
- Remove unused readFile import in cli/init.ts - Move SEED_INSTALL_COUNTS outside per-plugin loop in seed.ts - Fix membrain docs: remove fictitious Go 1.25+ requirement, remove non-standard transport field from MCP config example - Clarify SETUP.md: GitHub env vars don't come from Supabase dashboard Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 31babe3 commit 3de8197

4 files changed

Lines changed: 24 additions & 27 deletions

File tree

apps/cli/src/commands/init.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { readFile, writeFile, access } from "node:fs/promises";
1+
import { writeFile, access } from "node:fs/promises";
22
import { resolve, basename } from "node:path";
33
import chalk from "chalk";
44
import { confirm, input, checkbox, Separator } from "@inquirer/prompts";

apps/marketplace/SETUP.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ GITHUB_TOKEN=<github-pat>
2121
GITHUB_WEBHOOK_SECRET=<webhook-secret>
2222
```
2323

24-
All values are available in your Supabase project dashboard under **Settings → API**.
24+
Supabase values are in your project dashboard under **Settings → API**. GitHub values come from your repo webhook config and a GitHub PAT.
2525

2626
## Seed the database
2727

apps/marketplace/supabase/seed.ts

Lines changed: 20 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,26 @@ async function seedComponents(
187187
): Promise<void> {
188188
console.log(`Seeding ${plugins.length} components...`);
189189

190+
// Plausible seed install counts — real counts come from the install API route.
191+
const SEED_INSTALL_COUNTS: Record<string, number> = {
192+
research: 1240,
193+
review: 980,
194+
explain: 870,
195+
lineage: 640,
196+
orient: 590,
197+
capture: 520,
198+
"harness-share": 480,
199+
docgen: 430,
200+
"open-pr": 390,
201+
"merge-pr": 360,
202+
"pr-sweep": 310,
203+
stats: 290,
204+
membrain: 260,
205+
board: 230,
206+
"iterm-notify": 190,
207+
"frontend-design": 170,
208+
};
209+
190210
for (const plugin of plugins) {
191211
const skillMd = readSkillMds(plugin.name);
192212
const readmeMd = readReadmeMds(plugin.name);
@@ -197,28 +217,6 @@ async function seedComponents(
197217
.map((w) => w.charAt(0).toUpperCase() + w.slice(1))
198218
.join(" ");
199219

200-
// Assign plausible seed install counts by category tier.
201-
// These are rough estimates for initial seeding — real counts come from
202-
// production installs tracked by the install API route.
203-
const SEED_INSTALL_COUNTS: Record<string, number> = {
204-
research: 1240,
205-
review: 980,
206-
explain: 870,
207-
lineage: 640,
208-
orient: 590,
209-
capture: 520,
210-
"harness-share": 480,
211-
docgen: 430,
212-
"open-pr": 390,
213-
"merge-pr": 360,
214-
"pr-sweep": 310,
215-
stats: 290,
216-
membrain: 260,
217-
board: 230,
218-
"iterm-notify": 190,
219-
"frontend-design": 170,
220-
};
221-
222220
const component = {
223221
slug: plugin.name,
224222
name: displayName,

website/content/docs/plugins/research-knowledge/membrain.mdx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ Search, trace, and manage your [membrain](https://github.com/siracusa5/membrain)
1111

1212
## Requirements
1313

14-
- Go 1.25+
14+
- Go (any recent version)
1515

1616
```bash
1717
go install github.com/siracusa5/membrain/cmd/mem@latest
@@ -33,8 +33,7 @@ After install, add the membrain MCP server to your Claude Code settings:
3333
"mcpServers": {
3434
"membrain": {
3535
"command": "mem",
36-
"args": ["mcp"],
37-
"transport": "stdio"
36+
"args": ["mcp"]
3837
}
3938
}
4039
}

0 commit comments

Comments
 (0)