Fix SELECT * incorrectly included in aggregation-only queries

[Copilot]

The QueryASTBuilder was generating invalid AST by including a wildcard field (SELECT *) in queries that specified only aggregations without explicit fields. This is semantically incorrect—aggregation queries should select only the aggregated values.

Changes

• Modified buildSelect() to exclude the wildcard field when aggregations are present but no explicit fields are specified
• Changed condition from else to else if (!query.aggregations || query.aggregations.length === 0)

Before/After

// Query with only aggregations
const query = {
  object: 'orders',
  aggregations: [
    { function: 'count', alias: 'total_count' },
    { function: 'sum', field: 'amount', alias: 'total_amount' }
  ]
};

// Before: ast.select.fields = [{ type: 'field', name: '*' }, ...2 aggregations] ❌
// After:  ast.select.fields = [...2 aggregations] ✅

This aligns the AST generation with standard SQL semantics where SELECT COUNT(*), SUM(amount) does not include an additional wildcard field.

Original prompt

引用: https://github.com/objectstack-ai/objectui/actions/runs/21512259815/job/61983227116#step:8:1

---

💡 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.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
objectui	Ready	Preview, Comment	Jan 30, 2026 10:46am
objectui-components	Ready	Preview, Comment	Jan 30, 2026 10:46am
objectui-crm-app	Error		Jan 30, 2026 10:46am

[Copilot]

Pull request overview

This PR fixes a bug in the QueryASTBuilder where aggregation-only queries incorrectly included a wildcard field (SELECT *) in addition to the aggregation functions. The fix ensures that when a query specifies only aggregations without explicit fields, the generated AST contains only the aggregation fields, aligning with standard SQL semantics.

Changes:

• Modified the buildSelect() method to conditionally add the wildcard field only when both fields and aggregations are absent