refactor: refactoring core #2307
Open
+249,910
−2,771
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
nichenqin
changed the title
nichenqin
force-pushed
the
refactor/core
branch
from
045cfeb to
a823774
Compare
nichenqin
force-pushed
the
refactor/core
branch
3 times, most recently
from
8eb3533 to
e4bac47
Compare
nichenqin
force-pushed
the
refactor/core
branch
3 times, most recently
from
67db766 to
792f8d4
Compare
|
Deployment Status: ✅ Success |
nichenqin
force-pushed
the
refactor/core
branch
from
e517ca8 to
e2cb199
Compare
|
Deployment Status: ❌ Failed |
nichenqin
force-pushed
the
refactor/core
branch
from
e2cb199 to
5642f24
Compare
|
Deployment Status: ✅ Success |
nichenqin
force-pushed
the
refactor/core
branch
2 times, most recently
from
fe5003c to
a8b6afb
Compare
|
Deployment Status: ❌ Failed |
|
Deployment Status: ✅ Success |
|
Deployment Status: ❌ Failed |
1 similar comment
|
Deployment Status: ❌ Failed |
nichenqin
force-pushed
the
refactor/core
branch
from
e4a5d53 to
b36a914
Compare
|
Deployment Status: ✅ Success |
7 similar comments
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
…ground components with loading
…gurations for e2e tests
…mprove snapshot for fields
…es for FK constraints
nichenqin
force-pushed
the
refactor/core
branch
from
58b6835 to
3a70989
Compare
nichenqin
force-pushed
the
refactor/core
branch
from
3a70989 to
8389a7f
Compare
- Move leading space into orderByRef to avoid trailing space when empty - Update inline snapshots in ComputedFieldUpdater.spec.ts
…ate TypeScript configurations
… and analysis - Added `sql-formatter` dependency for SQL formatting. - Implemented `ExplainResultPanel` component for displaying SQL explain results. - Enhanced `RecordCreateDialog` and `RecordUpdateDialog` to include explain functionality. - Introduced command explain service and analyzers for create, update, and delete operations. - Updated HTTP contract to include explain endpoints for create, update, and delete commands. - Registered command explain module in the container node setup.
nichenqin
force-pushed
the
refactor/core
branch
from
0dba44f to
bef26b1
Compare
- Refactored CSS variables for improved color consistency and accessibility. - Introduced new background patterns and glassmorphism effects in styles. - Updated UI components in PlaygroundShell, ExplainResultPanel, and TableMetaPage for better visual hierarchy and responsiveness. - Enhanced DataTable with pinned column support and improved styling. - Added animations for various UI elements to enhance user experience. - Improved ScrollArea component to support horizontal and vertical scrolling.
Add comprehensive unit tests for all field schema rules using PGlite as an embedded PostgreSQL database. Tests verify: - up() creates expected schema elements and isValid() returns true - down() removes schema elements and isValid() returns false Rules tested: - ColumnExistsRule, ColumnUniqueConstraintRule, NotNullConstraintRule - FkColumnRule, ForeignKeyRule, IndexRule, UniqueIndexRule - OrderColumnRule, LinkValueColumnRule, GeneratedColumnRule - JunctionTableExistsRule and related rules - FieldMetaRule, ReferenceRule 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
… operator type inference - Add isConditionalRollupField predicate to fieldPredicates.ts - Add conditionalRollup handling in resolveColumnType (same as rollup) - Fix comparison/logical operators to always return single boolean value (not array even when comparing array fields - matches SQL layer behavior) - Enhance normalizeLinkRawValue to handle escaped quotes in JSON strings - Update all formula-sql-pg snapshots to reflect the fixes
- Added a new adapter for PostgreSQL table repository, including necessary configurations and types. - Implemented various utility functions and query builders to enhance database interactions. - Updated existing packages to integrate with the new adapter, ensuring compatibility and improved functionality. - Removed obsolete files and configurations from the previous adapter to streamline the codebase.
Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Introduced mechanisms in HybridWithOutboxStrategy and SyncInTransactionStrategy to track updated fields and filter them out in subsequent stages, preventing redundant updates. - Updated RecordUpdateBuilder to include computed update impact for better propagation planning. - Refactored related tests to ensure correct behavior of new update logic.
- Added exclusion patterns to the ESLint configuration to prevent linting of unnecessary directories. - Simplified type imports in ComputedUpdatePlanner for better readability. - Introduced unit tests for ComputedUpdatePlanner to validate update planning logic, ensuring correct behavior in various scenarios. - Adjusted ComputedUpdateWorker to handle change types more effectively during update stages.
- Added a new test to verify that the ComputedUpdateWorker correctly downgrades the changeType from 'insert' to 'update' during the planning of subsequent async stages. - Updated type imports for better clarity and consistency across the test files. - Adjusted type import in ComputedUpdateWorker to use named imports for improved readability.
…onents - Added a new context menu component using Radix UI for improved user interactions in the playground. - Updated various UI components to utilize the new context menu, including data tables and sidebar elements. - Enhanced styling and layout of the playground header, base page, and table cards for better visual consistency. - Introduced new dependencies for context menu functionality and updated existing packages to ensure compatibility.
- Introduced advisory locks for computed updates to manage concurrency, preventing conflicts during record updates. - Added configuration options for lock settings, including maximum record locks and lock modes. - Enhanced the ComputedFieldUpdater to acquire locks before executing update plans, ensuring serialized access to resources. - Updated explain command analyzers to include computed update lock information in the output. - Added utility functions to build and manage lock information for better integration with existing explain functionality. - Enhanced tests for the ComputedUpdateWorker and related components to validate lock behavior and ensure correct handling of concurrent updates.
- Implemented a dialog for setting and testing a PostgreSQL database URL in the Playground. - Introduced local storage management for persisting the database URL across sessions. - Enhanced the SchemaCheckPanel to utilize the database URL for schema validation. - Added API endpoint for checking database connection validity. - Updated UI components to reflect database connection status and provide user feedback. - Refactored related components to support the new database URL functionality.
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.
No description provided.