Skip to content

fix: decouple planexe-outputs from git repo - #230

Merged
neoneye merged 3 commits into
PlanExeOrg:mainfrom
VoynichLabs:fix/decouple-planexe-outputs
Mar 9, 2026
Merged

fix: decouple planexe-outputs from git repo#230
neoneye merged 3 commits into
PlanExeOrg:mainfrom
VoynichLabs:fix/decouple-planexe-outputs

Conversation

@82deutschmark

Copy link
Copy Markdown
Collaborator

Summary

This PR decouples the planexe-outputs directory from git version control to prevent accidental data loss from git reset operations.

Changes

  • Added planexe-outputs/ to .gitignore to prevent tracking
  • Made output directory configurable via PLANEXE_OUTPUTS_DIR environment variable
  • Defaults to run directory for backward compatibility

Motivation

This change prevents incidents where a git reset --hard could destroy live pipeline output directories. The planexe-outputs directory contains transient run data that should never be committed to git.

Testing

  • Backward compatible: existing deployments continue to use 'run' as default
  • New deployments can set PLANEXE_OUTPUTS_DIR=/path/to/outputs to customize location
  • Output directories are now properly ignored by git

- Add planexe-outputs/ to .gitignore to prevent accidental data loss
- Make output directory configurable via PLANEXE_OUTPUTS_DIR env var
- Default to 'run' directory for backward compatibility
- Prevents git reset from destroying pipeline output directories
@neoneye
neoneye merged commit 3978ddc into PlanExeOrg:mainMar 9, 2026
3 checks passed
@neoneye
neoneye deleted the fix/decouple-planexe-outputs branch March 9, 2026 21:32
neoneye added a commit that referenced this pull request Mar 9, 2026
* main:
fix: correct PLANEXE_OUTPUTS_DIR docs - accurate incident description
fix: address PR #230 review - add docs, remove gitignore change
fix: decouple planexe-outputs from git repo
docs: explain why ThinkingAwareOpenAILike exists vs LMStudio class
docs: Add thinking tokens section to LM Studio provider guide
Add ThinkingAwareOpenAILike class for LM Studio thinking token handling
docs(proposal-103): extend with llama_index silent truncation findings from local model runs
Sign up for freeto 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

@82deutschmark@neoneye