Commit graph

3 commits

Author SHA1 Message Date
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
2c6f8d560e test(unit): add comprehensive tests for value pluralism services
- PluralisticDeliberationOrchestrator: 38 tests (367 lines)
  - Framework detection (6 moral frameworks)
  - Conflict analysis and facilitation
  - Urgency tier determination
  - Precedent tracking
  - Statistics and edge cases

- AdaptiveCommunicationOrchestrator: 27 tests (341 lines)
  - Communication style adaptation (5 styles)
  - Anti-patronizing filter
  - Pub test validation (Australian/NZ)
  - Japanese formality handling
  - Statistics tracking

All 65 tests passing with proper framework keyword detection

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-12 16:35:30 +13:00