Fix TypeScript error: make LoggerConfig.name optional

[Copilot]

CI failing with Property 'name' is missing but required in packages/core/src/logger.ts. The LoggerConfigSchema defines name as optional, but the type constraint Required<Omit<LoggerConfig, 'file' | 'rotation'>> excluded only file/rotation, making name required.

Changes

• Type definition: Added 'name' to the Omit list to keep it optional
• Config initialization: Preserve name field if provided
• Pino integration: Pass name to Pino logger options when set

// Before
private config: Required<Omit<LoggerConfig, 'file' | 'rotation'>> & { ... };

// After  
private config: Required<Omit<LoggerConfig, 'file' | 'rotation' | 'name'>> & { 
  file?: string; 
  rotation?: { maxSize: string; maxFiles: number }; 
  name?: string 
};

The pattern now correctly handles all optional fields in LoggerConfigSchema.

Original prompt

引用: https://github.com/objectstack-ai/spec/actions/runs/21521817544/job/62014822275#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)
spec	Ready	Preview, Comment	Jan 30, 2026 5:14pm

[Copilot]

Pull request overview

This PR fixes a TypeScript compilation error where the name field in LoggerConfig was incorrectly required. The LoggerConfigSchema (in @objectstack/spec) defines name as optional, but the type constraint in the logger implementation was making it required by only excluding file and rotation from the Required type.

Changes:

• Updated the private config type to also omit 'name' from the Required constraint, making it properly optional
• Added name field preservation in config initialization
• Added conditional logic to pass name to Pino logger options when provided