Skip to content

fix(cli): type approvalFlow test helper's edges param (unblock TS on main)#2283

Merged
os-zhuang merged 1 commit into
mainfrom
fix/cli-lint-flow-test-implicit-any
Jun 24, 2026
Merged

fix(cli): type approvalFlow test helper's edges param (unblock TS on main)#2283
os-zhuang merged 1 commit into
mainfrom
fix/cli-lint-flow-test-implicit-any

Conversation

@os-zhuang

Copy link
Copy Markdown
Contributor

main's TypeScript Type Check is red: packages/cli/src/utils/lint-flow-patterns.test.ts:190error TS7006: Parameter 'edges' implicitly has an 'any' type (introduced by #2279). The TS job builds the whole workspace, so this blocks CI for every open PR (incl. the ADR renumber #2280).

One-line test-helper type annotation. No behavior change.

🤖 Generated with Claude Code

… on main)

`packages/cli/src/utils/lint-flow-patterns.test.ts:190` had an untyped `edges`
parameter (TS7006 under the build's noImplicitAny), introduced by #2279. The
TypeScript Type Check job builds the whole workspace, so this fails CI for
*every* open PR. Add the edge-array type (and an explicit approvalConfig type).
Test-only; no behavior change.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
spec Ready Ready Preview, Comment Jun 24, 2026 10:45am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

📓 Docs Drift Check

This PR changes 1 package(s): @objectstack/cli.

15 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:

  • content/docs/concepts/cloud-artifact-api.mdx (via packages/cli)
  • content/docs/concepts/implementation-status.mdx (via @objectstack/cli)
  • content/docs/concepts/packages.mdx (via @objectstack/cli)
  • content/docs/getting-started/cli.mdx (via @objectstack/cli)
  • content/docs/getting-started/quick-start.mdx (via @objectstack/cli)
  • content/docs/guides/authentication.mdx (via @objectstack/cli)
  • content/docs/guides/client-sdk.mdx (via @objectstack/cli)
  • content/docs/guides/hook-bodies.mdx (via packages/cli)
  • content/docs/guides/packages.mdx (via @objectstack/cli)
  • content/docs/guides/project-scoping.mdx (via @objectstack/cli)
  • content/docs/guides/runtime-services/data-service.mdx (via packages/cli)
  • content/docs/guides/runtime-services/index.mdx (via packages/cli)
  • content/docs/guides/skills.mdx (via packages/cli)
  • content/docs/protocol/objectos/plugin-spec.mdx (via @objectstack/cli)
  • content/docs/protocol/objectos/realtime-protocol.mdx (via @objectstack/cli)

Advisory only. To re-verify, run the docs-accuracy-audit workflow scoped to these files:
node scripts/docs-audit/affected-docs.mjs origin/main → pass the list as args.docs.

@os-zhuang
os-zhuang merged commit f415bd8 into main Jun 24, 2026
16 checks passed
@os-zhuang
os-zhuang deleted the fix/cli-lint-flow-test-implicit-any branch June 24, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant