Skip to content

Conversation

@sanity
Copy link
Collaborator

@sanity sanity commented Nov 19, 2025

Problem

Nacho (iduartgomez) raised portability concerns in review - AGENTS.md contained hardcoded paths like ~/code/freenet/freenet-core/main that aren't portable across development environments.

Solution

  • Replace hardcoded absolute paths with generic placeholders (<repo-root>/main)
  • Add import for ~/.claude/freenet-local.md to store machine-specific configuration
  • Reference freenet-test-network as GitHub repo instead of hardcoded local path
  • Simplify agent spawning template to be environment-agnostic

Structure

AGENTS.md (portable, checked into git):

  • Generic instructions using placeholders like <repo-root>/
  • Import statement: @~/.claude/freenet-local.md
  • Adaptable examples for different setups

~/.claude/freenet-local.md (machine-specific, NOT in git):

  • Actual paths for this machine (nova)
  • Local automation scripts and tooling
  • Build optimization settings

Testing

Created ~/.claude/freenet-local.md with nova-specific config and verified Claude Code correctly imports it when reading AGENTS.md.

Impact

Enables developers to use AGENTS.md as portable documentation while maintaining their own local configuration. Addresses review feedback without losing the detailed automation guidance.

[AI-assisted - Claude]

Changes:
- Replace hardcoded absolute paths with generic placeholders
- Add import for ~/.claude/freenet-local.md for machine-specific config
- Reference freenet-test-network as GitHub repo instead of hardcoded path
- Simplify agent spawning template to be environment-agnostic

The generic AGENTS.md now contains portable instructions, while
machine-specific paths and automation scripts can be configured in
~/.claude/freenet-local.md (not checked into git).

Addresses portability concerns raised in code review.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@sanity sanity enabled auto-merge November 19, 2025 19:53
@sanity sanity added this pull request to the merge queue Nov 19, 2025
Merged via the queue into main with commit 735de6d Nov 19, 2025
22 checks passed
@sanity sanity deleted the fix/agents-portability branch November 19, 2025 20:16
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