Skip to content

feat: flip napoln add default scope from global to project#106

Open
raiderrobert wants to merge 1 commit into
mainfrom
agent/feat/69-rethink-scope-semantics
Open

feat: flip napoln add default scope from global to project#106
raiderrobert wants to merge 1 commit into
mainfrom
agent/feat/69-rethink-scope-semantics

Conversation

@raiderrobert
Copy link
Copy Markdown
Owner

Implements #69.

See #69 for context.

What changed

  • Added napoln.core.project module with is_inside_project, load_config_default_scope, and resolve_scope helpers.
  • napoln add now defaults to project scope when run inside a recognized project (directory containing .git, .napoln, .claude, .agents, or .cursor).
  • Added --global / -g flag to napoln add for explicit global installation opt-in.
  • napoln add outside any project errors with a helpful hint suggesting --global.
  • Bootstrap skill (napoln-manage) is now always installed globally, regardless of command scope.
  • Default default_scope in new config.toml changed from "global" to "project".
  • Updated existing tests to be explicit about scope where they relied on the old global default.

How to verify

# Inside a git repo or project with agent dirs
uv run napoln add ./my-skill          # installs into project scope

# Outside any project
uv run napoln add ./my-skill          # errors with hint to use --global
uv run napoln add ./my-skill --global # installs globally

Implement #69.

- Add --global/-g flag to napoln add as explicit global opt-in
- Default napoln add to project scope when inside a project
- Error outside a project with helpful hint to use --global
- Add project detection via .git/.napoln/.claude/.agents/.cursor markers
- Resolve scope with priority: explicit flags > config > auto-detect
- Always install bootstrap skill (napoln-manage) globally
- Update default config default_scope from global to project
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