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 ISkillPackagepackages/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!
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
publishAtfield is written bypublishSharedSkillPackageand filtered by theworkspace-publicprovider (publishAt IS NOT NULL), but it was never declared as a column on theSkillPackageentity or theISkillPackagecontract. TypeORM silently drops the write, so the org market listing is always empty.Changes:
packages/contracts/src/ai/skill.model.ts— addpublishAt?: DatetoISkillPackagepackages/server-ai/src/skill-package/skill-package.entity.ts— add@Column({ type: 'timestamptz', nullable: true }) publishAt.changeset/fix-publishAt-column.md— patch changesetCould 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!