TheFlow
2298d36bed
fix(submissions): restructure Economist package and fix article display
...
- Create Economist SubmissionTracking package correctly:
* mainArticle = full blog post content
* coverLetter = 216-word SIR— letter
* Links to blog post via blogPostId
- Archive 'Letter to The Economist' from blog posts (it's the cover letter)
- Fix date display on article cards (use published_at)
- Target publication already displaying via blue badge
Database changes:
- Make blogPostId optional in SubmissionTracking model
- Economist package ID: 68fa85ae49d4900e7f2ecd83
- Le Monde package ID: 68fa2abd2e6acd5691932150
Next: Enhanced modal with tabs, validation, export
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-24 08:47:42 +13:00
TheFlow
f49bbe8455
refactor: remove orphaned tests for deleted website code
...
REMOVED: 15 test files testing non-existent code
Website Feature Tests (5):
- api.admin.test.js - Tests admin auth (auth.controller/routes removed)
- api.auth.test.js - Tests user authentication (auth.controller/routes removed)
- api.documents.test.js - Tests CMS documents (documents.controller/routes removed)
- api.koha.test.js - Tests donation system (koha.service/controller/routes removed)
- value-pluralism-integration.test.js - Website feature test
Removed Service Tests (5):
- BlogCuration.service.test.js - Service removed
- ClaudeAPI.test.js - Service removed
- koha.service.test.js - Service removed
- AdaptiveCommunicationOrchestrator.test.js - Service removed
- ProhibitedTermsScanner.test.js - Internal tool
Removed Util Tests (1):
- markdown.util.test.js - Util removed
Research/PoC Tests (4):
- tests/poc/memory-tool/* - Phase 5 proof-of-concept research
RETAINED: Framework service tests only
- BoundaryEnforcer, ContextPressureMonitor, CrossReferenceValidator
- InstructionPersistenceClassifier, MetacognitiveVerifier
- PluralisticDeliberationOrchestrator, MemoryProxy
- Integration tests for governance, projects, sync
REASON: Tests must test code that exists. Orphaned tests
provide false confidence and maintenance burden.
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-21 21:33:16 +13:00
TheFlow
1fe50500f0
feat(framework): implement Phase 1 proactive content scanning
...
CREATED:
- scripts/framework-components/ProhibitedTermsScanner.js (420 lines)
• Scans codebase for inst_016/017/018 violations
• Pattern detection for guarantee language, fabricated stats, unverified claims
• Auto-fix capability with context awareness
• CLI interface: --details, --fix, --staged flags
- tests/unit/ProhibitedTermsScanner.test.js (39 tests, all passing)
• Pattern detection tests (inst_017, inst_018)
• Context awareness tests
• Auto-fix functionality tests
• Edge case handling
MODIFIED:
- scripts/session-init.js
• Added Section 7: Scanning for Prohibited Terms
• Renumbered subsequent sections (CSP → 8, Dev Env → 9, Continuous → 10)
• Scans on every session start, reports violations
- scripts/hook-validators/validate-file-write.js
• Added missing checkPreActionCheckRecency() function (fixes hook crash)
- package.json/package-lock.json
• Added glob@11.0.3 dependency
RESULTS:
• Scanner operational: 39/39 tests passing
• Session integration: Runs automatically on session start
• Current scan: Found 364 violations (188 inst_017, 120 inst_018, 56 inst_016)
• Violations need user review (many in historical docs, specifications)
IMPACT:
• Framework now PROACTIVE instead of reactive
• Violations detected at session start (not weeks later)
• Auto-fix available for simple cases
• Closes critical detection gap identified in framework assessment
NEXT STEPS (user decision):
• Review 364 violations (many false positives in historical docs)
• Optionally: Implement pre-commit hook
• Phase 2: Context-aware rule surfacing
• Phase 3: Active metacognitive assistance
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-21 17:37:51 +13:00
