Skip to content

Config glob naming a directory lists the files directly inside it - #1358

Merged
thekevinbot merged 10 commits into
mainfrom
claude/1258-config-directory
Oct 7, 2026
Merged

thekevinbot merged 10 commits into
mainfrom
claude/1258-config-directory

Conversation

@thekevinbot

@thekevinbot thekevinbot commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Refs #1258.

A config [[table]] glob = "docs" (or "docs/") lists the files directly inside docs, as FROM './docs' does. Breaking. The directory is looked up under today's anchor (the index root); the lookup takes the root as a parameter so #1232 can feed it.

Checked by hand: docs, docs/ and top.md return identical rows on a config table and a path-table. Bash lists the directory itself for docs; a directory name listing its files is an approved divergence.

Red commit: c5220e9, CI run 37401216694.

Changelog / Migrations

  • Changelog fragment added under packages/{rust,python,ts}/changelog.d/
  • Migration fragment with the same slug added under packages/{rust,python,ts}/migrations.d/

E2E Verification

  • Ran e2e suites locally for every affected SDK
  • Python e2e receipt pending: main's Python e2e is red on glob_speed_depth_bound_test until Test one-level glob listings at speed of light #1317 merges; full-suite receipt will be written then.
  • TypeScript SDK e2e: pass
  • Rust core e2e: no new failures; watcher tests fail locally with Too many open files from the shared host, unrelated
  • packages/ts/e2e-attestations/<branch>.json receipt written
  • Command(s) run: just e2e-attest-ts
  • Result summary: TypeScript e2e attested; Python receipt pending as above.

Base automatically changed from claude/1258-config-gitignore to main October 6, 2026 23:52
@thekevinbot
thekevinbot merged commit 2f9456e into main Oct 7, 2026
83 checks passed
@thekevinbot
thekevinbot deleted the claude/1258-config-directory branch October 7, 2026 00:25
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.

1 participant