refactor and fixing mono repos - #1346
Closed
JayGhiya wants to merge 1319 commits into
Closed
Conversation
chore: release dev
chore: release dev
Update codebase artifacts (run 4bfec92b)
fix(optimise-framework-schema): move framework feature fields to JSON…
chore: release dev
…SONB feature payload
fix(optimise-framework-schema): switch ingestion and query paths to J…
feat(better-navigation-structure): add language-first DIDS catalog hi…
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix(links): link logo and globe icon to main website unoplat.io
chore: release dev
…erence Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix(irrelevant-link): remove FastAPI Catalog link from DIDS quick ref…
chore: release dev
…-file docs(agents): split dependency guides into dedicated overview files
chore: release dev
…ript fix(typescript): add concept enum in schema to support more typescript
…nize into typescript/ namespace Introduce monorepo and standalone detection adapters with workspace discovery utilities for Turbo, Nx, pnpm, yarn, and npm workspaces. Consolidate package_json_strategy and manifest loader under typescript/. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…aces Update strategy registry, parser, activities, detection utilities, dependency injection, router mappers, settings, and main entry point to import from the new python/, typescript/, and shared/ namespaces. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Delete old detectors/, pip/, poetry/, uv/, and utils/ directories whose code has been moved to the new python/, typescript/, and shared/ namespaces. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ule layout Fix import paths in existing strategy tests, expand TypeScript detection tests with test support infrastructure, relocate node/ tests and fixtures to typescript/, and remove the large turbo_monorepo test fixture. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Switch commons dependency to local path for development, add network test marker, update business logic references for new module paths, and refresh lockfile. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…e reorganization Delete the source files from detectors/, node/, pip/, utils/, and uv/ whose contents were moved to shared/, python/, and typescript/ namespaces. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ngine models and services Add workspace provenance and root fields to RepositoryRulesetMetadata, update repository metadata service and engineering workflow service to load and expose the new monorepo-aware detection fields. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…reorganization Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…-detector hierarchy Evolve DIDS from flat feature definitions to a 3-layer capability → operation → detector model, enabling architecture-level grouping of framework signatures.
…ierarchy Update the schema contribution guide to document the 3-layer capability → operation → detector model, grouping rules, and updated examples and troubleshooting.
…I definition Include fastapi.routing.APIRouter alongside fastapi.APIRouter for both http_endpoint and router_include features to fix incomplete detection.
Add explicit DocsPageLoaderData interface to route loader, track routeTree.gen.ts, and run sync:framework-definitions before typechecking.
Add task-24 family for architecture-level capability grouping, task-25 for docs build fixes, and doc-2 for Firebase TypeScript SDK discovery.
…ty-operation hierarchy - Upgrade schema.json from v3 to v4: replace flat 'features' with 'capabilities' → 'operations' hierarchy - Add capabilityFamilyEnum constraining allowed capability keys (http_endpoint, relational_database, llm_inference, etc.) - Migrate all Python definitions (fastapi, celery, fastmcp, litellm, pydantic, sqlalchemy, sqlmodel) to capability-operation structure - Migrate all TypeScript definitions (nextjs, litellm, swr, zustand) to capability-operation structure - Update README.md with v4 format docs and key concepts
…d FrameworkFeaturePayload - Add capability_key and operation_key optional fields to FeatureSpec - Add capability_key, operation_key, notes, and docs_url fields to FrameworkFeaturePayload - Change FrameworkFeaturePayload.model_config from extra='allow' to extra='forbid' for stricter validation - Fix import ordering in __init__.py files (alphabetical)
…re keys during ingestion - Update framework_loader to iterate capabilities→operations and produce composite 'capability.operation' feature keys - Inject capability_key and operation_key into each feature_definition payload before persistence - Update framework_query_service to extract capability_key and operation_key from feature_definition when building FeatureSpec
…ion keys - Refactor test_framework_definitions_ingestion for capability.operation composite keys across all Python and TypeScript definitions - Update test_framework_detection_with_postgres expected feature keys - Update test_framework_detection_language_processor assertions - Update test_framework_detection_tree_sitter expected keys and fixtures - Update test_framework_loader test data for v4 capabilities structure - Update test_nextjs_extraction for TypeScript operation keys - Add new test_framework_query_service for capability/operation extraction
…progress - Update task-24 description to reflect v4 adoption status - Update task-24.1, task-24.2, task-24.3 status - Add task-24.4 tree (24.4.1–24.4.5.3) for operation-key migration - Add task-26 for v4 capability taxonomy alignment
- Add engineering workflow section with MCP tool launch commands - Add CI testing commands (act) for local GitHub Actions runs
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…tion Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…nd agent definitions Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ll imports - Move activities into codebase_workflow_run/ and repository_workflow_run/ subpackages - Extract workflows into dedicated package with typed runners - Slim temporal_agents.py to registry facade delegating to agent_assembly - Remove search-mode branching from code_confluence_agents.py - Update imports in worker manager, workflow service, and interceptor - Delete old flat activity files and temporal_workflows.py Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ackage structure Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Important Review skippedToo many files! This PR contains 269 files, which is 119 over the limit of 150. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (31)
📒 Files selected for processing (269)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 119ec79d38
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -8,7 +6,7 @@ configs: | |||
| - value: true | |||
| constraints: {} | |||
| limit.maxIDLength: | |||
| - value: 255 | |||
| - value̋: 255 | |||
There was a problem hiding this comment.
value key for Temporal maxIDLength config
limit.maxIDLength now uses value̋ instead of value, so the Temporal dynamic-config entry is malformed. In deployments that use this compose file, Temporal will not receive the intended max ID limit (and may parse this entry incorrectly), which can cause runtime validation failures or unexpected defaults for workflow/run IDs. Please change this key back to value to keep production behavior aligned with the working local compose config.
Useful? React with 👍 / 👎.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.