@lobehub/lobehub 2.0.0-next.126 → 2.0.0-next.128

package/src/libs/redis/utils.test.ts

@@ -0,0 +1,46 @@
+import { describe, expect, it } from 'vitest';
+
+import {
+  buildIORedisSetArgs,
+  buildUpstashSetOptions,
+  normalizeMsetValues,
+  normalizeRedisKey,
+  normalizeRedisKeys,
+} from './utils';
+
+describe('redis utils', () => {
+  it('normalizes single redis key to string', () => {
+    expect(normalizeRedisKey('foo')).toBe('foo');
+    expect(normalizeRedisKey(Buffer.from('bar'))).toBe('bar');
+  });
+
+  it('normalizes an array of redis keys', () => {
+    expect(normalizeRedisKeys(['foo', Buffer.from('bar')])).toEqual(['foo', 'bar']);
+  });
+
+  it('normalizes mset values from Map', () => {
+    const payload = normalizeMsetValues(
+      new Map<Buffer | string, number>([
+        [Buffer.from('a'), 1],
+        ['b', 2],
+      ]),
+    );
+
+    expect(payload).toEqual({ a: 1, b: 2 });
+  });
+
+  it('builds ioredis set arguments', () => {
+    const args = buildIORedisSetArgs({ ex: 1, nx: true, get: true });
+
+    expect(args).toEqual(['EX', 1, 'NX', 'GET']);
+  });
+
+  it('builds upstash set options', () => {
+    expect(buildUpstashSetOptions()).toBeUndefined();
+    expect(buildUpstashSetOptions({ ex: 10, nx: true, get: true })).toEqual({
+      ex: 10,
+      nx: true,
+      get: true,
+    });
+  });
+});

package/src/libs/redis/utils.ts

@@ -0,0 +1,53 @@
+import type { SetCommandOptions } from '@upstash/redis';
+
+import { RedisKey, RedisMSetArgument, RedisValue, SetOptions } from './types';
+
+export const normalizeRedisKey = (key: RedisKey) =>
+  typeof key === 'string' ? key : key.toString();
+
+export const normalizeRedisKeys = (keys: RedisKey[]) => keys.map(normalizeRedisKey);
+
+export const normalizeMsetValues = (values: RedisMSetArgument): Record<string, RedisValue> => {
+  if (values instanceof Map) {
+    return Array.from(values.entries()).reduce<Record<string, RedisValue>>((acc, [key, value]) => {
+      acc[normalizeRedisKey(key)] = value;
+      return acc;
+    }, {});
+  }
+
+  return values;
+};
+
+export const buildIORedisSetArgs = (options?: SetOptions): Array<string | number> => {
+  if (!options) return [];
+
+  const args: Array<string | number> = [];
+
+  if (options.ex !== undefined) args.push('EX', options.ex);
+  if (options.px !== undefined) args.push('PX', options.px);
+  if (options.exat !== undefined) args.push('EXAT', options.exat);
+  if (options.pxat !== undefined) args.push('PXAT', options.pxat);
+  if (options.keepTtl) args.push('KEEPTTL');
+  if (options.nx) args.push('NX');
+  if (options.xx) args.push('XX');
+  if (options.get) args.push('GET');
+
+  return args;
+};
+
+export const buildUpstashSetOptions = (options?: SetOptions): SetCommandOptions | undefined => {
+  if (!options) return undefined;
+
+  const mapped: Partial<SetCommandOptions> = {};
+
+  if (options.ex !== undefined) mapped.ex = options.ex;
+  if (options.px !== undefined) mapped.px = options.px;
+  if (options.exat !== undefined) mapped.exat = options.exat;
+  if (options.pxat !== undefined) mapped.pxat = options.pxat;
+  if (options.keepTtl) mapped.keepTtl = true;
+  if (options.nx) mapped.nx = true;
+  if (options.xx) mapped.xx = true;
+  if (options.get) mapped.get = true;
+
+  return Object.keys(mapped).length ? (mapped as SetCommandOptions) : undefined;
+};

package/.github/workflows/check-console-log.yml

@@ -1,117 +0,0 @@
-name: Check Console Log (Warning)
-
-on:
-  pull_request:
-    branches:
-      - main
-      - next
-
-permissions:
-  contents: read
-  pull-requests: write
-
-jobs:
-  check-console-log:
-    name: Check for console.log statements (non-blocking)
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v5
-
-      - name: Install bun
-        uses: oven-sh/setup-bun@v2
-
-      - name: Run console.log check
-        id: check
-        run: |
-          OUTPUT=$(bunx tsx scripts/checkConsoleLog.mts 2>&1)
-          echo "$OUTPUT"
-
-          # Save output to file for later use
-          echo "$OUTPUT" > /tmp/console-log-output.txt
-
-          # Check if violations were found
-          if echo "$OUTPUT" | grep -q "Total violations:"; then
-            echo "has_violations=true" >> $GITHUB_OUTPUT
-            TOTAL=$(echo "$OUTPUT" | grep -oP "Total violations: \K\d+")
-            echo "total=$TOTAL" >> $GITHUB_OUTPUT
-          else
-            echo "has_violations=false" >> $GITHUB_OUTPUT
-          fi
-
-      - name: Comment on PR
-        if: steps.check.outputs.has_violations == 'true'
-        uses: actions/github-script@v7
-        env:
-          VIOLATION_COUNT: ${{ steps.check.outputs.total }}
-        with:
-          script: |
-            const fs = require('fs');
-            const output = fs.readFileSync('/tmp/console-log-output.txt', 'utf8');
-            const total = process.env.VIOLATION_COUNT || '0';
-
-            // Parse violations from output (format: "  file:line" followed by "    content")
-            const lines = output.split('\n');
-            const violations = [];
-            for (let i = 0; i < lines.length; i++) {
-              const line = lines[i];
-              // Match lines like "  packages/database/src/client/db.ts:258"
-              const fileMatch = line.match(/^\s{2}(\S+:\d+)\s*$/);
-              if (fileMatch) {
-                const file = fileMatch[1];
-                const content = lines[i + 1]?.trim() || '';
-                violations.push({ file, content });
-                i++;
-              }
-            }
-
-            // Build comment body
-            const maxDisplay = 30;
-            let body = `## ⚠️ Console.log Check Warning\n\n`;
-            body += `Found **${total}** \`console.log\` statement(s) in this PR.\n\n`;
-
-            if (violations.length > 0) {
-              body += `<details>\n<summary>📋 Click to see violations (${Math.min(violations.length, maxDisplay)} of ${total} shown)</summary>\n\n`;
-              body += `| File | Code |\n|------|------|\n`;
-              violations.slice(0, maxDisplay).forEach(v => {
-                const escapedContent = v.content
-                  .substring(0, 60)
-                  .replace(/\|/g, '\\|')
-                  .replace(/`/g, "'");
-                body += `| \`${v.file}\` | \`${escapedContent}${v.content.length > 60 ? '...' : ''}\` |\n`;
-              });
-              if (parseInt(total) > maxDisplay) {
-                body += `\n*...and ${parseInt(total) - maxDisplay} more violations*\n`;
-              }
-              body += `\n</details>\n\n`;
-            }
-
-            body += `> 💡 **Tip:** Remove \`console.log\` or add files to \`.console-log-whitelist.json\`\n`;
-            body += `> ✅ This check is **non-blocking** and won't prevent merging.`;
-
-            // Find existing comment to update instead of creating duplicates
-            const { data: comments } = await github.rest.issues.listComments({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: context.issue.number,
-            });
-
-            const botComment = comments.find(c =>
-              c.user.type === 'Bot' && c.body.includes('Console.log Check Warning')
-            );
-
-            if (botComment) {
-              await github.rest.issues.updateComment({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                comment_id: botComment.id,
-                body,
-              });
-            } else {
-              await github.rest.issues.createComment({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                issue_number: context.issue.number,
-                body,
-              });
-            }