Skip to content

Conversation

@praptisharma28
Copy link
Contributor

@praptisharma28 praptisharma28 commented Aug 17, 2025

Issues Fixed

This PR resolves several development blockers that prevent the Eremos framework from running out of the box:

1. Missing Development Dependencies

  • Problem: Repository uses TypeScript but lacks required devDependencies
  • Fix: Added typescript, ts-node, and @types/node to enable development
  • Impact: Developers can now run npm install and execute TypeScript code

2. Broken Script Imports

  • Problem: scripts/export-agent-memory.ts imports from "../agents" but no index file exists
  • Fix: Created agents/index.ts that exports all agents and provides a registry array
  • Impact: Scripts can now access agents programmatically without filesystem scanning

3. TypeScript Compilation Error

  • Problem: agents/launchtracker.ts passes confidence to logSignal() but the interface doesn't accept it
  • Error: TS2353: Object literal may only specify known properties, and 'confidence' does not exist
  • Fix: Added optional confidence?: number property to logSignal interface
  • Impact: Preserves the agent confidence scoring feature described in the README

4. Outdated / Incomplete README

  • Problem: The README lacked setup instructions and did not reflect the latest dev workflow
  • Fix: Refactored README with updated installation, development, and usage guidelines
  • Impact: New contributors can now set up and run the project smoothly without guesswork

Changes Made

New Files

  • agents/index.ts – Centralized agent registry and exports

Modified Files

  • package.json – Added TypeScript devDependencies
  • utils/logger.ts – Added confidence property to logSignal interface
  • README.md – Updated with clear setup and usage instructions

Testing

# Install dependencies
npm install

# Verify TypeScript compilation
npx tsc --noEmit

# Test agent imports work
npx ts-node -e "import('./agents/index').then(m => console.log('Loaded', m.agents.length, 'agents'))"

# Test previously broken script
npx ts-node scripts/export-agent-memory.ts

- Add missing TypeScript devDependencies (typescript, ts-node, @types/node)
- Create agents/index.ts to fix broken import references in scripts
- Add confidence property to logSignal interface to support agent scoring
- Enables npm install && basic development workflow out of the box

Fixes compilation error in agents/launchtracker.ts and broken imports
in scripts/export-agent-memory.ts
@praptisharma28 praptisharma28 changed the title fix: resolve TypeScript errors and missing development dependencies fix: resolve TypeScript errors, missing development dependencies and refactor README Aug 18, 2025
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.

1 participant