Skip to content

feat: porting over python to typescript #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brendancopley
Copy link
Owner

feat: porting over python to typescript

@brendancopley brendancopley requested a review from Copilot March 22, 2025 18:58
@brendancopley brendancopley self-assigned this Mar 22, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Port Python functionality to a TypeScript codebase with new infrastructure for linting, analytics, reasoning, formatting, and example handling.

  • Introduces new ESLint configuration, TypeScript modules for reasoning, analytics and complexity estimation, and corresponding tests.
  • Sets up new endpoints via an Express server and a Docker Compose file for containerized deployment.

Reviewed Changes

Copilot reviewed 27 out of 30 changed files in this pull request and generated no comments.

Show a summary per file
File Description
eslint.config.ts New ESLint configuration utilizing TypeScript parser, plugins, and rule spreads.
src/typescript/index.ts Entry point re-exporting new TypeScript modules.
src/typescript/reasoning.ts Logic for selecting reasoning approach and creating prompts.
src/typescript/tests/client.test.ts Tests for client completions, chat, complexity, and analytics recording.
src/typescript/format.ts Utility to enforce and analyze step formatting.
src/typescript/server.ts Express server providing endpoints for completions, chat, models, and analytics.
src/typescript/prompts.ts Functions for creating prompts for reasoning.
src/typescript/examples.ts Example database management and example transformation functions.
docker-compose.yml Container orchestration configuration for the app and PostgreSQL database.
src/typescript/client.ts Main client combining reasoning, formatting, and analytics functionalities.
src/typescript/analytics.ts Analytics service providing inference recording and performance statistics.
src/typescript/complexity.ts Complexity estimator module analyzing problems and calculating complexity scores.
Files not reviewed (3)
  • .prettierrc: Language not supported
  • prisma/schema.prisma: Language not supported
  • write_files.sh: Language not supported
Comments suppressed due to low confidence (2)

eslint.config.ts:43

  • The configuration is spreading properties from 'eslint.configs.recommended.rules', but the imported '@typescript-eslint/eslint-plugin' may not expose a 'configs' property. Verify that the correct configuration object is referenced or import the intended recommended configuration directly.
      ...eslint.configs.recommended.rules,

src/typescript/examples.ts:40

  • Typo in the reasoning text: 'Let'\s' should be corrected to "Let's" to maintain proper spelling and readability.
        reasoning: "Let'\s think through this step by step:\n1. Initially, Jason had 20 lollipops."

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