Fix DTS build: duplicate export and AnalyticsQuery type mismatch

[Copilot]

DTS build fails with TS2308 (duplicate AnalyticsQueryRequest export) and TS2416 (MemoryAnalyticsService.query incompatible with IAnalyticsService). Root cause: analytics.zod.ts and protocol.zod.ts both export AnalyticsQueryRequest, and the contracts AnalyticsQuery interface diverged from the Zod schema.

• src/api/protocol.zod.ts — Remove export from 4 analytics type aliases that duplicate exports already in analytics.zod.ts. Kept as local types for IObjectStackProtocol interface usage.
• src/contracts/analytics-service.ts — Align AnalyticsQuery with AnalyticsQuerySchema (Zod source of truth): cube → optional, measures → required.
• packages/plugins/driver-memory/src/memory-analytics.ts — Add cube presence guard in query() and generateSql() now that the type reflects optionality.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
objectstack-play	Ready	Preview, Comment	Feb 12, 2026 5:45am
spec	Ready	Preview, Comment	Feb 12, 2026 5:45am

[Copilot]

Pull request overview

This pull request fixes TypeScript build errors related to duplicate exports and type misalignment in the analytics module. The root cause was that both analytics.zod.ts and protocol.zod.ts exported the same type names, causing TS2308 errors, and the AnalyticsQuery contract interface diverged from its Zod schema source of truth.

Changes:

• Removed duplicate export keywords from 4 analytics type aliases in protocol.zod.ts, converting them to non-exported types used only within that file's ObjectStackProtocol interface
• Aligned the AnalyticsQuery contract interface with the Zod schema: made cube optional and measures required, with documentation explaining the 2-layer architecture pattern
• Added runtime guards in MemoryAnalyticsService.query() and generateSql() to validate cube presence, since the type now reflects optionality

Reviewed changes

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

File	Description
packages/spec/src/api/protocol.zod.ts	Changed 4 analytics type aliases from exported to non-exported (lines 1060-1063) to avoid duplicate exports with analytics.zod.ts
packages/spec/src/contracts/analytics-service.ts	Made cube optional and measures required in AnalyticsQuery interface to align with AnalyticsQuerySchema Zod definition
packages/plugins/driver-memory/src/memory-analytics.ts	Added cube presence validation in query() (lines 59-61) and generateSql() (lines 271-273) methods

[Copilot]

The type AnalyticsResultResponse is referenced in the ObjectStackProtocol interface but is not exported from analytics.zod.ts, where the corresponding schema AnalyticsResultResponseSchema is defined.

Add the missing export to analytics.zod.ts:

export type AnalyticsResultResponse = z.infer<typeof AnalyticsResultResponseSchema>;

This export should be added alongside the other analytics type exports (after line 84 in analytics.zod.ts) to maintain consistency and allow external code to import this type when needed.

Suggested change

	type AnalyticsQueryRequest = z.infer<typeof AnalyticsQueryRequestSchema>;
	type AnalyticsResultResponse = z.infer<typeof AnalyticsResultResponseSchema>;
	type GetAnalyticsMetaRequest = z.infer<typeof GetAnalyticsMetaRequestSchema>;
	type GetAnalyticsMetaResponse = z.infer<typeof AnalyticsMetadataResponseSchema>;
	export type AnalyticsQueryRequest = z.infer<typeof AnalyticsQueryRequestSchema>;
	export type AnalyticsResultResponse = z.infer<typeof AnalyticsResultResponseSchema>;
	export type GetAnalyticsMetaRequest = z.infer<typeof GetAnalyticsMetaRequestSchema>;
	export type GetAnalyticsMetaResponse = z.infer<typeof AnalyticsMetadataResponseSchema>;

[Copilot]

The type GetAnalyticsMetaRequest is referenced in the ObjectStackProtocol interface but is not exported from analytics.zod.ts, where the corresponding schema GetAnalyticsMetaRequestSchema is defined.

Add the missing export to analytics.zod.ts:

export type GetAnalyticsMetaRequest = z.infer<typeof GetAnalyticsMetaRequestSchema>;

This export should be added alongside the other analytics type exports to allow external code to import this type when needed.

Suggested change

	type GetAnalyticsMetaRequest = z.infer<typeof GetAnalyticsMetaRequestSchema>;
	export type GetAnalyticsMetaRequest = z.infer<typeof GetAnalyticsMetaRequestSchema>;

[Copilot]

There's a naming inconsistency: the schema is AnalyticsMetadataResponseSchema and the exported type in analytics.zod.ts is AnalyticsMetadataResponse, but here you're using the name GetAnalyticsMetaResponse.

For consistency, either:

1. Use AnalyticsMetadataResponse (the already-exported name from analytics.zod.ts), OR
2. Add an export for GetAnalyticsMetaResponse as an alias in analytics.zod.ts

Option 1 is recommended to match the existing export pattern in analytics.zod.ts line 85.

Suggested change

	type GetAnalyticsMetaResponse = z.infer<typeof AnalyticsMetadataResponseSchema>;
	type AnalyticsMetadataResponse = z.infer<typeof AnalyticsMetadataResponseSchema>;