PR #681 closed without review — clarification needed

[Strongf-bob]

Hi maintainers,

PR #681 (fix: add missing publishAt column to SkillPackage entity and contract) was closed earlier today without any review or comment.

This PR fixes a production issue: the publishAt field is written by publishSharedSkillPackage and filtered by the workspace-public provider (publishAt IS NOT NULL), but it was never declared as a column on the SkillPackage entity or the ISkillPackage contract. TypeORM silently drops the write, so the org market listing is always empty.

Changes:

• packages/contracts/src/ai/skill.model.ts — add publishAt?: Date to ISkillPackage
• packages/server-ai/src/skill-package/skill-package.entity.ts — add @Column({ type: 'timestamptz', nullable: true }) publishAt
• .changeset/fix-publishAt-column.md — patch changeset

Could you let me know why it was closed? Happy to address any feedback or adjust the PR (e.g., target a different base branch).

Thanks!

[tiven-w]

First of all, we apologize for directly cherry-picking part of your code, creating a separate PR, and then closing your original PR.
Secondly, the publishAt field is defined in the parent type IBasePerWorkspaceEntityModel of ISkillPackage, and the corresponding entity class also inherits this definition.
Therefore, the issue you encountered may have been caused by something else. Could you please try to reproduce the problem again and let us know if it still occurs?