Skip to content

fix(dynamodb): improve item creation and result pagination - #545

Merged
Blankll merged 3 commits into
masterfrom
codex/dynamodb-create-item-usability
Nov 13, 2026
Merged

Blankll merged 3 commits into
masterfrom
codex/dynamodb-create-item-usability

Conversation

@Blankll

@Blankll Blankll commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Keep primary key fields and footer actions visible while the attribute list scrolls. Arrange optional index keys in a responsive grid that shows each field's type and secondary indexes.
  • Preserve drafts in memory per connection and table. Enable editing as soon as schema metadata arrives.
  • Add accessible field labels, validation feedback, keyboard access to the structured value editor, and Simplified and Traditional Chinese labels.
  • Preserve exact numeric literals in text-encoded lists, maps, and number sets through the DynamoDB PutItem request.
  • Show a centered loading overlay during item creation and a brief checkmark after success. Keep the current result page visible under the loading overlay during pagination and preserve it for retry if a request fails.
  • Apply the selected page size to fetched PartiQL batches before requesting the next cursor. A batch of 28 rows with a page size of 25 now displays 25 rows on the first page and three on the second.

Verification

  • npm run lint:check and npm run build
  • npx jest --runInBand (1,982 passed)
  • cargo test --manifest-path src-tauri/Cargo.toml --lib (778 passed)
  • DynamoDB Local item-flow integration test
  • Chromium smoke checks for light and dark themes, short panes, keyboard editing, item submission, pagination retry, and a 28-row PartiQL result with and without a server cursor

@codecov

codecov Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.95146% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.25%. Comparing base (c112325) to head (884608a).

Files with missing lines Patch % Lines
.../dynamo-editor/composables/usePartiqlPagination.ts 70.58% 8 Missing and 7 partials ⚠️
src-tauri/src/common/dynamodb_utils.rs 86.66% 8 Missing ⚠️
src/store/dbDataStore.ts 77.77% 1 Missing and 5 partials ⚠️
...tor/dynamo-editor/composables/useDynamoItemForm.ts 93.10% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #545      +/-   ##
==========================================
+ Coverage   71.88%   72.25%   +0.36%     
==========================================
  Files         169      170       +1     
  Lines       29735    29675      -60     
  Branches     1479     1506      +27     
==========================================
+ Hits        21376    21441      +65     
+ Misses       7909     7772     -137     
- Partials      450      462      +12     
Files with missing lines Coverage Δ
src-tauri/src/capabilities/dynamo.rs 96.78% <100.00%> (+0.12%) ⬆️
src/lang/enUS.ts 100.00% <ø> (ø)
src/lang/zhCN.ts 100.00% <ø> (ø)
src/lang/zhTW.ts 100.00% <ø> (ø)
...rc/views/editor/dynamo-editor/composables/index.ts 100.00% <100.00%> (ø)
...tor/dynamo-editor/composables/useDynamoItemForm.ts 96.61% <93.10%> (-3.39%) ⬇️
src/store/dbDataStore.ts 85.23% <77.77%> (+6.46%) ⬆️
src-tauri/src/common/dynamodb_utils.rs 92.15% <86.66%> (-0.46%) ⬇️
.../dynamo-editor/composables/usePartiqlPagination.ts 70.58% <70.58%> (ø)

... and 73 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Blankll Blankll changed the title fix(dynamodb): harden the Create Item workflow fix(dynamodb): improve item creation and result pagination Nov 12, 2026
@Blankll
Blankll merged commit d55ad67 into master Nov 13, 2026
9 checks passed
@Blankll
Blankll deleted the codex/dynamodb-create-item-usability branch November 13, 2026 02:16
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