Skip to content

[codex] Add Redis Array support#5861

Draft
rowantrollope wants to merge 1 commit intomainfrom
feature/add-redis-arrays-support
Draft

[codex] Add Redis Array support#5861
rowantrollope wants to merge 1 commit intomainfrom
feature/add-redis-arrays-support

Conversation

@rowantrollope
Copy link
Copy Markdown

What

Adds end-to-end Redis Array support in Browser, including backend Array APIs, key-info handling, Redis command registration, create/read/search/edit/delete UI flows, Array key styling/filtering, and validation for unsigned 64-bit array indexes.

Also renders escaped Markdown-like Array values as Markdown in the Array details table so stored documentation/text content is readable.

Testing

  • git diff --check
  • ./node_modules/.bin/jest 'src/modules/browser/array/array.service.spec.ts' 'src/modules/browser/keys/key-info/strategies/array.key-info.strategy.spec.ts' 'src/modules/browser/keys/key-info/key-info.provider.spec.ts' -w 1
  • ./node_modules/.bin/jest 'redisinsight/ui/src/utils/tests/formatters/markdownText.spec.ts' 'redisinsight/ui/src/utils/tests/validations.spec.ts' -c 'jest.config.cjs' -t 'markdownText|validateArrayIndex'
  • ./node_modules/.bin/eslint --quiet redisinsight/ui/src/pages/browser/modules/key-details/components/array-details/array-details-table/ArrayDetailsTable.tsx redisinsight/ui/src/utils/formatters/markdownText.ts redisinsight/ui/src/utils/tests/formatters/markdownText.spec.ts redisinsight/api/src/modules/browser/array/array.service.ts redisinsight/api/src/modules/browser/array/dto/get.array-elements.response.ts redisinsight/api/src/modules/browser/array/array.service.spec.ts
  • yarn build:api
  • yarn build:ui

No linked issue was provided.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented May 5, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Code Coverage - Backend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 92.27% 15689/17004
🟡 Branches 74.05% 4847/6546
🟢 Functions 86.07% 2416/2807
🟢 Lines 92.09% 14999/16288

Test suite run success

3371 tests passing in 307 suites.

Report generated by 🧪jest coverage report action from 39d403a

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Code Coverage - Integration Tests

Status Category Percentage Covered / Total
🟡 Statements 79.29% 17628/22231
🟡 Branches 61.86% 8049/13011
🟡 Functions 67.59% 2422/3583
🟡 Lines 78.85% 16580/21027

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 81.52% 24253/29752
🟡 Branches 67.14% 10202/15195
🟡 Functions 76.69% 6538/8525
🟢 Lines 81.97% 23705/28919

Test suite run success

6782 tests passing in 787 suites.

Report generated by 🧪jest coverage report action from 39d403a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants