Skip to content

feat: import shared indico agent guidance #14

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

Open
moliholy wants to merge 7 commits into
base: master
Choose a base branch
from
Open

feat: import shared indico agent guidance #14

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
3aa1859
feat: add indico-agents submodule and shared symlinks
moliholy May 25, 2026
afe3c8a
chore: bump indico agents submodule and add claude bridge
moliholy May 29, 2026
6bef788
chore: bump indico agents submodule
moliholy May 29, 2026
3aa4e32
Add Makefile target to install agent skills
OmeGak Sep 3, 2026
dbb348a
Update README.md
OmeGak Sep 3, 2026
7909b29
chore: broaden agent entries in `.gitignore`
moliholy Sep 4, 2026
ffe7fd3
chore: bump indico agents submodule and relink shared docs
moliholy Sep 4, 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
4 changes: 4 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,7 @@ webpack-build-config.json

# local settings
.envrc.local

# per-contributor AI assistant links (installed via agents/indico/scripts/install-links.sh)

@OmeGak OmeGak Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's keep this comment more general. Those directories may contain stuff other than links. Also, let's avoid references to paths that may change.

Suggested change
# per-contributor AI assistant links (installed via agents/indico/scripts/install-links.sh)
# local agent settings

@moliholy moliholy Sep 4, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Applied.

/.agents/skills/

@OmeGak OmeGak Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why ignoring only .agents/skill/ and not .agents/?

@moliholy moliholy Sep 4, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No good reason, .agents/ is now ignored whole. Nothing under it is committed: the installer only creates skill symlinks there, and .claude points at it.

/.claude

@OmeGak OmeGak Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's stick to the same syntax as in the rest of the file.

Suggested change
/.agents/skills/
/.claude
.agents/skills/
.claude

@moliholy moliholy Sep 4, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Applied the syntax, but widened the first entry to .agents/ per your other comment.

3 changes: 3 additions & 0 deletions .gitmodules
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,6 @@
[submodule "plugins/indico-plugins-contrib"]
path = plugins/indico-plugins-contrib
url = git@github.com:indico/indico-plugins-contrib.git
[submodule "agents/indico"]
path = agents/indico
url = https://github.com/unconventionaldotdev/indico-agents.git
1 change: 1 addition & 0 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
agents/indico/AGENTS.md
2 changes: 2 additions & 0 deletions CLAUDE.md
View file
Open in desktop

@OmeGak OmeGak Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why not making this file also a symlink to a CLAUDE.md file in the indico-agents submodule?

@moliholy moliholy Sep 4, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done. CLAUDE.md now lives in the submodule and the installer symlinks it like every other shared document, so the redirect has a single source of truth. Claude resolves the @AGENTS.md import relative to the file that declares it, so the same target serves both the root and indico/. Submodule side in unconventionaldotdev/indico-agents#2.

Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# Claude Code entrypoint. Redirects to the shared cross-agent guidance.
@AGENTS.md
1 change: 1 addition & 0 deletions CODING_GUIDELINES.md
View file
Open in desktop

@OmeGak OmeGak Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Anything against naming this file CONVENTIONS.md?

@moliholy moliholy Sep 4, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Nothing against it, renamed to CONVENTIONS.md. The document covers coding, testing, style, git and PR conventions, so the broader name fits better. Submodule side in unconventionaldotdev/indico-agents#2.

Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
agents/indico/CODING_GUIDELINES.md
1 change: 1 addition & 0 deletions agents/indico
Open in desktop
Submodule indico added at f2850f