Skip to content

Commit 9c18bce

Browse files
committed
chore(sync): cascade fleet template@764ec50
Auto-applied by socket-wheelhouse sync-scaffolding into vscode-socket-security. 2 file(s) touched: - .claude/hooks/marketplace-comment-guard/README.md - .claude/hooks/marketplace-comment-guard/test/index.test.mts
1 parent 4f971df commit 9c18bce

2 files changed

Lines changed: 17 additions & 15 deletions

File tree

.claude/hooks/marketplace-comment-guard/README.md

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,14 +41,16 @@ against — opaque pins look fine and stay broken for months.
4141

4242
```markdown
4343
<!-- .claude-plugin/README.md -->
44-
| plugin | version | sha | date | by |
45-
|--------|---------|------------------------------------------|------------|---------|
46-
| codex | v1.0.1 | 9cb4fe4099195b2587c402117a3efce6ab5aac78 | 2026-05-18 | jdalton |
44+
| plugin | version | sha | date | notes |
45+
|--------|---------|------------------------------------------|------------|----------------------------------|
46+
| codex | v1.0.1 | 9cb4fe4099195b2587c402117a3efce6ab5aac78 | 2026-05-18 | upstream openai/codex-plugin-cc |
4747
```
4848

49-
The first four columns are required and inspected. Extra columns (`by`,
50-
free-form `notes`) are accepted but not validated — those are human
51-
metadata.
49+
The first four columns are required and inspected. Any trailing column
50+
(e.g. free-form `notes`) is accepted but not validated. `git blame` is the
51+
authoritative record of *who* bumped a pin, so a `by` column is deliberately
52+
absent — duplicating personal identifiers into fleet-canonical files is a
53+
public-surface-hygiene mistake.
5254

5355
## What's enforced
5456

@@ -64,7 +66,7 @@ metadata.
6466

6567
- The accuracy of `date` — that's a human-review concern (same as the
6668
GHA `uses:` rule).
67-
- The `by` / `notes` columns — free-form metadata.
69+
- Any trailing `notes` column — free-form metadata.
6870
- Source types other than `git-subdir` carrying a `ref` field — if you
6971
add a new source type that doesn't have `ref`, the guard skips that
7072
entry rather than blocking. Add explicit support if the new type

.claude/hooks/marketplace-comment-guard/test/index.test.mts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ const VALID_JSON = JSON.stringify(
5656

5757
const VALID_README = `# marketplace
5858
59-
| plugin | version | sha | date | by |
60-
|--------|---------|------------------------------------------|------------|---------|
61-
| codex | v1.0.1 | ${SHA} | 2026-05-18 | jdalton |
59+
| plugin | version | sha | date | notes |
60+
|--------|---------|------------------------------------------|------------|-------|
61+
| codex | v1.0.1 | ${SHA} | 2026-05-18 | test |
6262
`
6363

6464
test('SKIPS non-marketplace paths', () => {
@@ -142,8 +142,8 @@ test('BLOCKS Write of marketplace.json when README version is stale', () => {
142142
test('BLOCKS Write of marketplace.json when README has no row for a plugin', () => {
143143
const noRowReadme = `# marketplace
144144
145-
| plugin | version | sha | date | by |
146-
|--------|---------|-----|------|-----|
145+
| plugin | version | sha | date | notes |
146+
|--------|---------|-----|------|-------|
147147
`
148148
const { dir, jsonPath } = makeFixture(null, noRowReadme)
149149
try {
@@ -207,7 +207,7 @@ test('BLOCKS Edit of README that removes a plugin row', () => {
207207
tool_name: 'Edit',
208208
tool_input: {
209209
file_path: readmePath,
210-
old_string: `| codex | v1.0.1 | ${SHA} | 2026-05-18 | jdalton |\n`,
210+
old_string: `| codex | v1.0.1 | ${SHA} | 2026-05-18 | test |\n`,
211211
new_string: '',
212212
},
213213
})
@@ -230,8 +230,8 @@ test('ALLOWS Edit of README that bumps a row in sync with a JSON bump (simulated
230230
file_path: readmePath,
231231
// No-op edit (replacing a string with itself) — content stays
232232
// consistent with on-disk JSON.
233-
old_string: 'jdalton',
234-
new_string: 'jdalton',
233+
old_string: 'test',
234+
new_string: 'test',
235235
},
236236
})
237237
assert.equal(exitCode, 0)

0 commit comments

Comments
 (0)