feat(graph+supervisor): layout dependency edges and graph-backed dependencies in validate_code#68
Open
fklosowski wants to merge 16 commits into
Open
feat(graph+supervisor): layout dependency edges and graph-backed dependencies in validate_code#68fklosowski wants to merge 16 commits into
fklosowski wants to merge 16 commits into
Conversation
…re adapter - Re-export `getPosition` utility for reuse in sibling modules. - Update `assembleResult` to accept and handle dependencies. - Implement `runStructure` to resolve outgoing dependencies using platformos-graph. - Add integration tests for dependency mapping in structure adapter. - Ensure linting and dependency resolution run concurrently in `validate_code`.
fklosowski
changed the title
…from GraphQL queries - Implemented extractGraphqlTable function to extract platformOS model table names from GraphQL operations. - Added unit tests for extractGraphqlTable covering various scenarios including shorthand and nested table filters. - Introduced GraphQL table extraction in the build process for schema modules. - Enhanced AppGraph and AppModule types to include schema and table properties. - Created fixtures for testing GraphQL queries with and without table filters. - Updated traversal logic to handle schema nodes and their associated table names.
- Updated index.ts to export `extractStructural` alongside `extractFileReferences`. - Introduced `GraphBuildOptions` in types.ts to control structural fact inclusion during builds. - Created adapter-input.ts to standardize input for lint and structure adapters. - Refactored lint.ts to utilize the new AdapterInput interface. - Enhanced assembleResult to include structural snapshots in the result. - Updated structure.ts to derive both dependencies and structural data from the same parse. - Added tests for validate-code to ensure lint and structure orchestration works as expected. - Improved error handling in validate-code to log structure resolution failures without dropping lint diagnostics.
- Updated DocumentsLocator to support canonical asset paths for both app and module assets. - Added tests to verify asset resolution under app/assets and module public/assets. - Introduced new asset edge handling in graph traversal and query logic. - Refactored asset module retrieval to use resolved URIs directly. - Improved integration tests to reflect changes in asset path resolution.
- Removed the `ValidateCodeDependency` and `ValidateCodeStructuralSnapshot` types and their associated logic. - Introduced `ValidateCodeImpact` type to represent cross-file blast radius. - Updated `assembleResult` function to accept and process impact data instead of dependencies and structural snapshots. - Modified `runStructure` to be removed, and its functionality replaced with impact analysis. - Adjusted `runValidateCode` to orchestrate linting and impact analysis, ensuring lint failures take precedence. - Updated tests to reflect changes in the impact analysis and ensure correct behavior. - Removed obsolete structure tests and integrated impact tests into the validation flow.
…hance GraphCache for fresh serving
- Add `deserializeAppGraph` function to reconstruct an in-memory AppGraph from its serialized form, ensuring round-trip identity and incremental readiness. - Introduce unit tests for `deserializeAppGraph` to validate serialization and deserialization processes, including edge cases for dangling edges and entry points. - Create a new module for graph cache persistence, including encoding and decoding cache files with versioning to handle corrupt or incompatible caches. - Enhance `GraphCache` to support loading from a persisted cache on cold starts, allowing for faster initialization and incremental updates. - Update server initialization to utilize the new cache path for improved performance and reliability.
…rove signature risk reporting
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.