Remove individual LLM schemas #219
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request removes individual LLM schema implementations and related type checkers, consolidating the codebase to use a unified ILlmSchema approach. It also reorganizes the directory structure by flattening the composers folder hierarchy.
Key Changes
- Removed vendor-specific schema types (ChatGPT, Claude, Gemini) and their version-specific variants (LlmSchemaV3, LlmSchemaV3_1)
- Removed corresponding type checkers for deleted schema types
- Flattened the composers directory structure by moving files out of
llm/andmigrate/subdirectories - Updated all import paths to reflect the new flat structure
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/*.ts (TypeCheckers) | Deleted 7 type checker files for individual LLM implementations |
| src/structures/*.ts (Schemas) | Deleted 5 schema structure files for specific LLM vendors and versions |
| src/composers/llm/*.ts (Composers) | Deleted 5 composer files for individual schema types |
| src/composers/*.ts (Moved files) | Moved LlmParametersComposer and LlmDescriptionInverter from llm/ subfolder with updated imports |
| src/composers/*.ts (Migrate files) | Moved 3 HttpMigrate* files from migrate/ subfolder with updated imports |
| src/composers/LlmSchemaComposer.ts | Updated imports to reflect flattened directory structure |
| src/HttpMigration.ts | Updated import path for HttpMigrateApplicationComposer |
| src/index.ts | Removed exports for deleted schema types, type checkers, and validation structures while retaining core exports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
samchon
added a commit
that referenced
this pull request
Dec 23, 2025
* Unified to `ILlmSchema`, no more separation. (#213) * Unified to `ILlmSchema` * Complete the new `ILlmSchema` type * detailed description comments * Update src/structures/ILlmSchema.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/structures/ILlmSchema.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/structures/ILlmSchema.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * prettier --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Unify `ILlmApplication` and `ILlmFunction` too. (#214) * Universal `LlmSchemaComposer` (#215) * Universal LlmSchemaComposer * complete `LlmSchemaComposer.schema()` function * fix logics * Universal `LlmTypeChecker` (#216) * Universal `LlmTypeChecker` * Fix `LlmSchemaComposer` to utilize `LlmTypeCheckeer` * JSDoc comments on universal LLM types. (#217) * Universal `HttpLlm` (#218) * Universal `HttpLlm` * Update src/HttpLlm.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/HttpLlm.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix configuration comments * fix more thing --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Remove individual LLM schemas (#219) * Remove individual LLM schemas * fix * Unify test functions about LLM schemas (#220) * Unify test functions about LLM schemas * Update test/src/utils/LlmApplicationFactory.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fixed most of test functions * fixed most of test functions * completed everything --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Publish v6 * Re-write README for universal LLM schemas (#221) --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request primarily reorganizes the codebase by flattening the
composersdirectory structure, moving several files out of subfolders, and cleaning up imports accordingly. Additionally, it removes two unused files:ClaudeSchemaComposer.tsandChatGptSchemaComposer.ts. No functional code changes were made, only file moves and import path updates.Key changes include:
Directory Restructuring and File Moves:
src/composers/migrate/tosrc/composers/:HttpMigrateApplicationComposer.tsHttpMigrateRouteAccessor.tsHttpMigrateRouteComposer.tssrc/composers/llm/tosrc/composers/:LlmDescriptionInverter.tsLlmParametersComposer.tsImport Path Updates:
Code Cleanup:
ClaudeSchemaComposer.tsandChatGptSchemaComposer.tsfromsrc/composers/llm/. [1] [2]LlmSchemaComposer.tsandLlmSchemaV3Composer.ts. [1] [2]These changes help simplify the project structure and make the codebase easier to navigate.