Fix Redis driver pipeline parsing and runtime/server test initialization

[Copilot]

Fixes two issues preventing CI from passing:

1. Redis Driver Pipeline Result Parsing

The Redis driver bulkUpdate and bulkDelete operations were parsing pipeline results using the v3 client API format [error, result], but the project uses v4 which returns results directly.

bulkUpdate:

• Access pipeline GET results directly instead of via tuple indexing: getResults?.[i] vs result?.[1]
• Guard against empty pipeline execution when no records exist

bulkDelete:

• Filter DEL results directly: r > 0 vs r && r[1] > 0

Example

// Before (v3 API assumption)
const result = getResults?.[i] as any;
const existingData = result?.[1];  // Assumes [error, result] tuple

// After (v4 API)
const existingData = getResults?.[i];  // Direct result access

The v4 client returns [1, 1, 0] for three DEL operations (two successful, one non-existent key), not [[null, 1], [null, 1], [null, 0]].

2. Runtime/Server Test Initialization

Added missing await app.init() calls to test setup in 4 test files that were failing because ObjectQL v4.0 requires initialization before use:

• node.test.ts
• rest.test.ts
• graphql.test.ts
• rest-advanced.test.ts

These failures were previously hidden by pnpm's "stop on first failure" behavior until the Redis driver tests were fixed.

Original prompt

引用: https://github.com/objectstack-ai/objectql/actions/runs/21284280204/job/61261497154#step:10:1

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
objectql-site	Ready	Preview, Comment	Jan 23, 2026 0:01am

[github-actions]

⚠️ No Changeset Found

This PR does not include a changeset file.
If this PR includes user-facing changes, please add a changeset by running:

pnpm changeset

[hotlong]

@copilot fix test

[Copilot]

@copilot fix test

The Redis driver tests that were originally failing are now fixed (commit 0b8bc17). However, the CI is now showing failures in packages/runtime/server tests that were previously hidden because pnpm stops at the first test failure.

These server test failures appear to be pre-existing issues from the v4.0 merge (#172), not caused by my Redis driver changes. Would you like me to investigate and fix the runtime/server test failures as well?

[github-actions]

⚠️ No Changeset Found

This PR does not include a changeset file.
If this PR includes user-facing changes, please add a changeset by running:

pnpm changeset

[Copilot]

Pull request overview

This PR fixes Redis pipeline result parsing to align with the Redis v4 client API and ensures ObjectQL runtime/server tests correctly initialize the app before use so CI can pass.

Changes:

• Updated RedisDriver.executeCommand bulk operations (bulkUpdate, bulkDelete) to treat multi().exec() results as direct replies instead of [error, result] tuples, and added a safety guard to avoid executing empty pipelines.
• Added missing await app.init() calls in REST, REST advanced, GraphQL, and Node server tests so handlers are created only after the ObjectQL app is fully initialized.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/drivers/redis/src/index.ts	Adjusts Redis pipeline result handling for bulkUpdate and bulkDelete to match the v4 client API and guards setPipeline.exec() when no updates are enqueued.
packages/runtime/server/test/rest.test.ts	Ensures the REST adapter tests call await app.init() before creating the REST handler and HTTP server.
packages/runtime/server/test/rest-advanced.test.ts	Ensures advanced REST/error-handling tests initialize the ObjectQL app before building the handler and server.
packages/runtime/server/test/node.test.ts	Ensures Node adapter tests call await app.init() after schema registration so requests run against an initialized app.
packages/runtime/server/test/graphql.test.ts	Ensures GraphQL adapter tests initialize the ObjectQL app before creating the GraphQL handler and server.

Comments suppressed due to low confidence (2)

packages/runtime/server/test/node.test.ts:13

• Unused import IObjectQL.

import { IObjectQL, Driver } from '@objectql/types';

packages/drivers/redis/src/index.ts:336

• This use of variable 'filters' always evaluates to true.

        if (filters && !Array.isArray(filters) && filters.filters) {