Skip to content

Include the .agents folder during publish - #34

Merged
Piedone merged 3 commits into
mainfrom
ma/fix-deploy-ci
Aug 23, 2026
Merged

Piedone merged 3 commits into
mainfrom
ma/fix-deploy-ci

Conversation

@MikeAlhayek

@MikeAlhayek MikeAlhayek commented Aug 22, 2026 •

Copy link
Copy Markdown
Member

This pull request updates the deployment workflow to ensure that the .agents/skills directory (which contains agent skills) is correctly included in the published and deployed package. Previously, hidden files and folders like .agents/skills could be silently dropped during artifact upload, resulting in deployments missing required skills. The changes add steps to explicitly preserve .agents, remove all other hidden files, and verify that agent skills are present both before and after artifact upload.

Artifact packaging and verification improvements:

  • Added a step to remove all hidden files and folders from the publish output except for .agents, ensuring only the required hidden directory is shipped.
  • Enabled the include-hidden-files: true option for artifact upload to allow .agents to be included in the package.
  • Added a verification step after pruning to confirm that at least one agent skill (SKILL.md) exists in .agents/skills before uploading the artifact.

Deployment package validation:

  • Added a guard step after artifact download to verify that agent skills are still present in the deployment package, preventing deployments with missing skills.

@MikeAlhayek
MikeAlhayek requested a review from Piedone August 22, 2026 18:09
@Piedone

Piedone commented Aug 22, 2026

Copy link
Copy Markdown
Member

This looks like something that needs to be fixed in the packages, no? Because otherwise every consumer needs special care for publishing. You can do such copying from a package's .targets file.

@MikeAlhayek

Copy link
Copy Markdown
Member Author

No. The package already includes it. The problem is that the deploy pipeline ignores or does not include the hidden folders like the package included .agents folder. So it does not publish it.

We need the pipeline to include it

@Piedone

Piedone commented Aug 23, 2026

Copy link
Copy Markdown
Member

Okay. One issue though, that this will also upload any skills we may add to to this repo for local development as well.

@Piedone
Piedone merged commit ef62db4 into main Aug 23, 2026
3 checks passed
@Piedone
Piedone deleted the ma/fix-deploy-ci branch August 23, 2026 00:26
@MikeAlhayek

Copy link
Copy Markdown
Member Author

Yes. It will. I doubt this website will ever need skills though.

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