Skip to content

Add publishAt regression coverage for shared skill packages#692

Draft
meta-d with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-publishat-column-issue
Draft

Add publishAt regression coverage for shared skill packages#692
meta-d with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-publishat-column-issue

Conversation

Copilot AI commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

workspace-public depends on publishAt being persisted and queryable on SkillPackage; if that field falls out of the contract or entity metadata, shared skill listings can silently disappear. This PR adds focused regression coverage around that path and records the patch in a changeset.

  • Contracts

    • Add a focused spec for ISkillPackage to keep publishAt visible on the shared skill package shape.
  • Entity metadata

    • Add a TypeORM metadata spec for SkillPackage to assert that the inherited publishAt column remains present with the expected timestamptz / nullable configuration.
  • Release note

    • Add a patch changeset describing the publishAt protection for workspace-public publishing flows.
const publishAtColumn = getMetadataArgsStorage()
  .filterColumns([SkillPackage, WorkspaceBaseEntity])
  .find(({ propertyName }) => propertyName === 'publishAt')

expect(publishAtColumn).toMatchObject({
  propertyName: 'publishAt',
  options: expect.objectContaining({
    type: 'timestamptz',
    nullable: true
  })
})

Copilot AI changed the title [WIP] Fix missing publishAt column in SkillPackage entity and contract Add publishAt regression coverage for shared skill packages Jun 22, 2026
Copilot AI requested a review from meta-d June 22, 2026 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants