Skip to content

Conversation

@jeherve
Copy link

@jeherve jeherve commented Jan 23, 2026

Fixes #337

Add configurable plans directory via SUPERPOWERS_PLANS_DIR environment variable

Motivation and Context

Currently, plan documents are always saved to docs/plans/ within the project directory. This can clutter the repository with planning documents that may not need to be committed. This change allows users to configure an external location for saving plans, keeping them separate from the codebase while still organizing them by project.

How Has This Been Tested?

Manual testing with SUPERPOWERS_PLANS_DIR set and unset to verify both code paths work correctly.

Breaking Changes

None. The default behavior remains unchanged - plans are still saved to docs/plans/ when SUPERPOWERS_PLANS_DIR is not set.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

When SUPERPOWERS_PLANS_DIR is set, plans are saved to a project-specific subdirectory based on the current working directory path relative to ~. For example, working in ~/code/myorg/webapp with SUPERPOWERS_PLANS_DIR=~/Documents/plans saves to ~/Documents/plans/code/myorg/webapp/YYYY-MM-DD-<feature-name>.md. The directory is created automatically if it doesn't exist.

Summary by CodeRabbit

  • New Features

    • Plan save location is now configurable via environment variable, with automatic directory creation.
    • Default behavior preserved when variable is not set.
  • Documentation

    • Updated guidance with examples for configurable plan locations.

✏️ Tip: You can customize this high-level summary in your review settings.

Allow users to specify a custom location for saving plan documents
using the SUPERPOWERS_PLANS_DIR environment variable. When set,
plans are saved to a project-specific subdirectory based on the
current working directory path.
@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

Plan file save locations are now configurable via the SUPERPOWERS_PLANS_DIR environment variable. Two skill documentation files were updated to implement this functionality with fallback to the default docs/plans/ directory location when the variable is not set.

Changes

Cohort / File(s) Change Summary
Plan Location Configuration
skills/brainstorming/SKILL.md, skills/writing-plans/SKILL.md
Updated plan save logic to reference configurable path based on SUPERPOWERS_PLANS_DIR environment variable. If set, saves to $SUPERPOWERS_PLANS_DIR/<project-path>/YYYY-MM-DD-<feature-name>.md with automatic directory creation; otherwise preserves default docs/plans/ behavior. Expanded guidance with path resolution examples.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Plans now wander where you choose,
No longer bound to rigid docs,
SUPERPOWERS_PLANS_DIR lights the path—
Freedom in folders, at long last! 📁✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main feature: adding a configurable plans directory via an environment variable, which is the primary change in this PR.
Linked Issues check ✅ Passed The PR fully implements the requested feature from #337: adds configurable plan save location via SUPERPOWERS_PLANS_DIR environment variable with fallback to default docs/plans/ behavior.
Out of Scope Changes check ✅ Passed All changes are scoped to implementing the configurable plans directory feature: updates to two SKILL.md files document the new environment variable behavior with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Make plan file save location configurable

1 participant