Skip to content

feat: Change hardcoded path variables in core modules#1

Closed
sashavdv wants to merge 1 commit into
l33tdawg:mainfrom
sashavdv:feat/path-variable-is-generalized
Closed

feat: Change hardcoded path variables in core modules#1
sashavdv wants to merge 1 commit into
l33tdawg:mainfrom
sashavdv:feat/path-variable-is-generalized

Conversation

@sashavdv

Copy link
Copy Markdown

Changed the hardcoded path variables in core modules so they work on every computer.

l33tdawg added a commit that referenced this pull request Apr 7, 2026
Aether now learns from every audit. SAGE provides persistent institutional
memory that reduces false positives and improves finding quality over time.

Core modules:
- sage_client.py: Thread-safe REST singleton for SAGE API
- sage_seeder.py: Pre-trained knowledge seeder with 170 JSON fixtures
  (75 exploit patterns, 63 archetype checklists, 12 token quirks, 20 historical exploits)
- sage_feedback.py: Feedback loop connecting audit outcomes to SAGE

Pipeline integration:
- Deep analysis: SAGE recall in pre-pipeline, Pass 3, Pass 5; store learnings post-pipeline
- Validation pipeline: Stage -1 SAGE known FP check filters previously rejected patterns
- Accuracy tracker: Forwards submission outcomes to SAGE feedback manager
- Audit runner: Post-audit SAGE remember/reflect in finally block

Infrastructure:
- docker-compose.yml for SAGE container (ghcr.io/l33tdawg/sage:latest)
- Auto-seed on first launch via background thread
- TUI CostBar shows SAGE ON/OFF status with memory count
- Config: sage_enabled/sage_url in ~/.aether/config.yaml

Community contributions integrated:
- @sashavdv: Fixed hardcoded path variables (PR #1)
- @pro258b: Added missing validate_anthropic_key() (PR #2)

90 new tests (2171 total), 10 E2E improvement tests proving marked
FP reduction and cross-audit knowledge transfer.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@l33tdawg

l33tdawg commented Apr 7, 2026

Copy link
Copy Markdown
Owner

Thanks @sashavdv! Great catch on the hardcoded paths. We've integrated your fix into the v5.0 release — all three hardcoded /Users/l33tdawg/ paths are now using Path.home(). You're credited in the release notes. Closing this as the fix is included in v5.0.

@l33tdawg

l33tdawg commented Apr 7, 2026

Copy link
Copy Markdown
Owner

Integrated into v5.0 — thanks for the contribution!

@l33tdawg l33tdawg closed this Apr 7, 2026
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