Skip to content

Correct the map kind spellings in AGENTS.md - #36

Merged
necco-c merged 1 commit into
masterfrom
fix-agents-map-kinds
Oct 8, 2026
Merged

necco-c merged 1 commit into
masterfrom
fix-agents-map-kinds

Conversation

@necco-c

@necco-c necco-c commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Fixes the kind list in template/AGENTS.md (line 273), which yeet new copies into every new project.

Problem

Five of the eight kinds listed are spellings the daemon rejects: hash-map, lru-hash-map, percpu-*, lpm-trie and bloom-filter. The daemon accepts only the underscore and no-separator forms (translate_bind_spec in crates/daemon/src/core/blobs/manager/mod.rs). Anything else is UnknownKind, but what reaches the script is No service for map.

We ran two cold coding agents inside a fresh yeet new project on yeet 0.23.4. Both followed this line, bound an LRU map as lru-hash-map, got No service for map, and found lru_hash by trial and error.

Change

  • Before: ringbuf, hash-map, lru-hash-map, array, percpu-*, lpm-trie, bloom-filter, data
  • After: ringbuf, hash, lru_hash, array, percpu_hash, lru_percpu_hash, percpu_array, lpm_trie, bloom_filter, data, plus one sentence saying hyphenated forms are rejected and show up as No service for map

Only those lines change; the rest of the paragraph is as it was. Every other kind: in the template (ringbuf, data, array) was already correct.

Why merging is enough

yeet new clones this repo's master when it scaffolds, so new projects get the fix as soon as this merges, with no yeet release.

Related, not in this PR

The daemon detects a misspelled kind, but the UnknownKind error never reaches the script; it surfaces as No service for map. Passing the real error through would make this mistake obvious.

The list used hyphenated kinds (hash-map, lru-hash-map, percpu-*,
lpm-trie, bloom-filter). The daemon accepts only the underscore and
no-separator forms (translate_bind_spec in
crates/daemon/src/core/blobs/manager/mod.rs), so an agent that follows
this file gets "No service for map". Both cold-agent runs scaffolded
with yeet new on 2026-10-08 hit it.
@julian-goldstein
julian-goldstein self-requested a review October 8, 2026 14:43
@necco-c
necco-c merged commit 677b1ca into master Oct 8, 2026
1 check 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