Skip to content

fix: add missing .describe() annotations and subpath exports for cloud/qa/identity/shared#687

Merged
hotlong merged 2 commits into
mainfrom
copilot/send-patch
Feb 15, 2026
Merged

fix: add missing .describe() annotations and subpath exports for cloud/qa/identity/shared#687
hotlong merged 2 commits into
mainfrom
copilot/send-patch

Conversation

Copilot AI commented Feb 15, 2026

Copy link
Copy Markdown
Contributor

Eight Zod fields lacked .describe() annotations, and four modules exported from root were missing build entry points and subpath exports.

Missing .describe() annotations

  • field.zod.tssummaryOperations.object, .field, .function
  • object.zod.tsObjectExtensionSchema.label, .pluralLabel, .description, .validations, .indexes

Missing subpath exports

cloud, qa, identity, shared were namespace-exported from src/index.ts but absent from tsup.config.ts entry points and package.json exports map. This meant they were bundled into the root chunk only — no standalone subpath import was possible.

// Previously broken — module not resolved
import * as Cloud from '@objectstack/spec/cloud';

// Now works
import * as Cloud from '@objectstack/spec/cloud';
import * as QA from '@objectstack/spec/qa';
import * as Identity from '@objectstack/spec/identity';
import * as Shared from '@objectstack/spec/shared';

All 5,376 tests pass. Build produces correct output for all 16 entry points.

Original prompt

发一个补丁


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Feb 15, 2026
@vercel

vercel Bot commented Feb 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
objectstack-play Ready Ready Preview, Comment Feb 15, 2026 4:20am
spec Canceled Canceled Feb 15, 2026 4:20am

Request Review

…d, qa, identity, shared modules

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
@hotlong
hotlong marked this pull request as ready for review February 15, 2026 04:03
Copilot AI review requested due to automatic review settings February 15, 2026 04:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR completes the build infrastructure and documentation for the @objectstack/spec package by adding missing tsup entry points, package.json subpath exports, and Zod .describe() annotations.

Changes:

  • Added 4 missing tsup entry points (cloud, qa, identity, shared) to enable proper module bundling
  • Added 4 corresponding package.json subpath exports to enable imports like @objectstack/spec/cloud
  • Added .describe() annotations to 8 previously undocumented Zod schema fields for JSON Schema generation

Reviewed changes

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

File Description
packages/spec/tsup.config.ts Added entry points for cloud, qa, identity, and shared modules to complete the build configuration
packages/spec/package.json Added subpath exports for cloud, qa, identity, and shared to enable modular imports
packages/spec/src/data/field.zod.ts Added .describe() annotations to 3 summaryOperations fields (object, field, function) for documentation
packages/spec/src/data/object.zod.ts Added .describe() annotations to 5 ObjectExtensionSchema fields (label, pluralLabel, description, validations, indexes) for documentation

Copilot AI changed the title [WIP] Send a patch to improve functionality fix: add missing .describe() annotations and subpath exports for cloud/qa/identity/shared Feb 15, 2026
Copilot AI requested a review from hotlong February 15, 2026 04:05
@hotlong
hotlong merged commit 4e23a75 into main Feb 15, 2026
7 of 9 checks passed
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.

3 participants