jonah/safe-ai-tools
1
0
Fork 0
Code Issues Pull requests 2 Projects Releases Packages Wiki Activity Actions

docs: point AGENTS.md at the vault hub #5

Merged
jonah merged 2 commits from docs/vault-hub into main 2026-10-06 15:11:41 +02:00
Conversation 2 Commits 2 Files changed 1 +5
jonah commented 2026-10-06 14:29:51 +02:00
Owner
Copy link

Adds a ## Documentation pointer to the vault hub (Engineering/Projects/safe-ai-tools/00 Overview.md), as the vault conventions ask for a new repo. Outside the Esper-managed block so a regenerate keeps it. Docs only, no version bump.

🤖 Generated with Claude Code

Adds a `## Documentation` pointer to the vault hub (`Engineering/Projects/safe-ai-tools/00 Overview.md`), as the vault conventions ask for a new repo. Outside the Esper-managed block so a regenerate keeps it. Docs only, no version bump. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
docs: point AGENTS.md at the vault hub
All checks were successful
CI / python (pull_request) Successful in 17s
Details
CI / checks (pull_request) Successful in 1m8s
Details
CI / policy (pull_request) Successful in 3s
Details
51746c3b08 
The hub is Engineering/Projects/safe-ai-tools/00 Overview.md. Placed outside
the Esper-managed block so a regenerate does not remove it.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
jonah commented 2026-10-06 14:29:52 +02:00
Author
Owner
Copy link

🤖 Auto-review status · jonah/safe-ai-tools PR #5

✅ PR is clean — no fixes required.

Last updated: 2026-10-06T12:35:32.163Z

<!-- opencode-status --> **🤖 Auto-review status** · `jonah/safe-ai-tools` PR #5 ✅ **PR is clean** — no fixes required. *Last updated: 2026-10-06T12:35:32.163Z*
jonah reviewed 2026-10-06 14:35:32 +02:00
jonah left a comment
Copy link

[STATUS: CLEAN]

Review scope: origin/main...HEAD — 1 commit (51746c3 docs: point AGENTS.md at the vault hub), 1 file changed, +5 lines (AGENTS.md:54-57).

Findings:

  • No bugs, security, or correctness issues. Docs-only change; no code, config, or CI paths touched.
  • Placement is correct: the new ## Documentation section sits after <!-- esper:project-context:end -->, so esper-cli init won't overwrite it (AGENTS.md:52-57).
  • Gating checks pass: git diff --check clean, trailing newline present, file at 57/500 lines, no secrets, gitleaks/end-of-file-fixer unaffected.
  • Pointer propagates to Claude Code via CLAUDE.md → @AGENTS.md.

Non-blocking notes (no action required):

  • AGENTS.md:56 — the vault pointer depends on the obsidian MCP with no fallback, unlike the Esper section (AGENTS.md:15-17) which explicitly degrades when its MCP is down. Harmless today; add a plain-path fallback only if agents actually stall on missing docs.
  • AGENTS.md:55 — "plans ... live in the vault, not here" is loosely contradicted by BRIEF.md living in-repo. Wording nit only; product docs (README, CHANGELOG) are clearly out of scope of the sentence.
[STATUS: CLEAN] **Review scope:** `origin/main...HEAD` — 1 commit (`51746c3 docs: point AGENTS.md at the vault hub`), 1 file changed, +5 lines (`AGENTS.md:54-57`). **Findings:** - No bugs, security, or correctness issues. Docs-only change; no code, config, or CI paths touched. - Placement is correct: the new `## Documentation` section sits *after* `<!-- esper:project-context:end -->`, so `esper-cli init` won't overwrite it (`AGENTS.md:52-57`). - Gating checks pass: `git diff --check` clean, trailing newline present, file at 57/500 lines, no secrets, `gitleaks`/`end-of-file-fixer` unaffected. - Pointer propagates to Claude Code via `CLAUDE.md` → `@AGENTS.md`. **Non-blocking notes (no action required):** - `AGENTS.md:56` — the vault pointer depends on the `obsidian` MCP with no fallback, unlike the Esper section (`AGENTS.md:15-17`) which explicitly degrades when its MCP is down. Harmless today; add a plain-path fallback only if agents actually stall on missing docs. - `AGENTS.md:55` — "plans ... live in the vault, not here" is loosely contradicted by `BRIEF.md` living in-repo. Wording nit only; product docs (README, CHANGELOG) are clearly out of scope of the sentence.
Merge branch 'main' into docs/vault-hub
Some checks failed
CI / checks (pull_request) Successful in 1m6s
Details
CI / policy (pull_request) Failing after 2s
Details
CI / python (pull_request) Successful in 18s
Details
5c33465303
jonah merged commit 9172286d48 into main 2026-10-06 15:11:41 +02:00
jonah deleted branch docs/vault-hub 2026-10-06 15:11:41 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jonah/safe-ai-tools!5
No description provided.