Skip to content

feat: agent folders hierarchy — hierarchical agent folder structure (JAC-4764) #11081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
JackReis wants to merge 24 commits into from
+10,501 −721
Closed

feat: agent folders hierarchy — hierarchical agent folder structure (JAC-4764) #11081

Show file tree
Hide file tree
Changes from 23 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
b1e3095
fix(task-watchdogs): key human-wait fingerprint on kind/presence, not…
JackReis Jul 31, 2026
032f67f
fix(task-watchdogs): align noop stopped classification
JackReis Jul 31, 2026
4e038c8
fix: deduplicate .claude/skills/design-guide via symlink (matches mai…
JackReis Aug 4, 2026
6d9c517
ci: re-trigger after flaky tmp-cleanup ENOTEMPTY in adapter-claude-lo…
JackReis Aug 5, 2026
f85a255
feat(JAC-4747): Add agent_folders table and folder_id column for Phas…
JackReis Aug 8, 2026
efacc47
feat(JAC-4746/JAC-4747): agent-folders hierarchy — routes, services, …
JackReis Aug 8, 2026
e7646a7
feat(JAC-4754): flat-to-folder migration tooling — CLI commands, rout…
JackReis Aug 8, 2026
d369d56
feat(JAC-4753): CLI Commands for Folder Management (Phase 3)
JackReis Aug 8, 2026
92ed13b
feat(JAC-4750): Instruction Inheritance Resolution Engine — core impl…
JackReis Aug 8, 2026
9c8bbe0
feat(JAC-4752/Phase 3): pointer files + agent integration
JackReis Aug 8, 2026
6d7fc03
feat(JAC-4746): Complete Phase 3 — folder instructions-bundle endpoin…
JackReis Aug 8, 2026
321da68
feat(JAC-4896): JAC-4748 Phase 2 adapter integration for merged instr…
JackReis Aug 8, 2026
d9b39d3
test(JAC-4755): folder instruction-inheritance integration tests + docs
JackReis Aug 8, 2026
036e806
feat(agent-folders): load _CONTEXT.md (HFS room) into agent prompt vi…
JackReis Aug 8, 2026
193c0a5
docs(JAC-5087): Phase 1 folder CRUD verification receipt
JackReis Aug 8, 2026
086ee2a
docs(JAC-4864): Phase 1 folder CRUD verification artifact
JackReis Aug 8, 2026
39ce825
feat(JAC-4751): Integrate hierarchical folder instructions into claud…
JackReis Aug 8, 2026
0bb620e
feat(JAC-4751): Integrate hierarchical folder instructions into Herme…
JackReis Aug 8, 2026
005ceee
fix(JAC-4821): make folder-migration tests compatible with FK constra…
JackReis Aug 8, 2026
f5747ff
test(JAC-4754): commit Phase 4 agent-folders integration test suites
JackReis Aug 8, 2026
090a4a8
fix(db): remove duplicate migration journal entry for 0195_jac4538
JackReis Aug 8, 2026
fe96ce3
fix: resolve merge conflict in agent-folders-routes-integration.test.ts
JackReis Aug 8, 2026
223cb0c
fix(test): use unique issue_prefix C3 in company isolation test to av…
JackReis Aug 8, 2026
9b6ae19
fix(JAC-4764): remove committed merge conflict markers from agent-fol…
JackReis Aug 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .claude/skills/design-guide
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
../../.agents/skills/design-guide

@superagent-security superagent-security Bot Aug 8, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2: Skill content removed and replaced with unreviewed external symlink

Design-guide skill content (674 lines) deleted and replaced with a symlink to an external, unreviewed path, unrelated to the stated PR scope.

Revert the symlink and keep skill content version-controlled, or document the security rationale and ensure the target is reviewed.

AI prompt 
Check if this security scanner issue is valid. If so, understand the root cause and fix it. If appropriate, update or add tests. Keep the change focused and preserve intended behavior.

<file name=".claude/skills/design-guide">
<violation number="1" location=".claude/skills/design-guide:1">
<priority>P2</priority>
<title>Skill content removed and replaced with unreviewed external symlink</title>
<evidence>The PR removes the entire `.claude/skills/design-guide/` directory contents (SKILL.md with 351 lines and references/component-index.md with 323 lines) and replaces it with a symlink file pointing to `../../.agents/skills/design-guide`. This change is not mentioned in the PR title or body, which describes an unrelated 'agent folders hierarchy' feature. By replacing version-controlled skill content with a symlink to an external path, future modifications to the skill would no longer be visible in PR diffs, bypassing review controls.</evidence>
<recommendation>Either keep skill content fully version-controlled in the repository, or document the symlink change in the PR description with a clear security rationale and ensure the symlink target is also under version control and review. Revert the symlink if it was added unintentionally.</recommendation>
</violation>
</file>

351 changes: 0 additions & 351 deletions .claude/skills/design-guide/SKILL.md
View file
Open in desktop

This file was deleted.

Loading
Loading