[dotnet-port-fixes] Align skill asset blocks with .NET

[michelle-clayton-work]

Port the .NET skill content behavior that emits explicit available resources and scripts blocks for file-based skills.

• [dotnet-port-fixes] Align skill asset blocks with .NET #377

[Copilot]

Pull request overview

This PR aligns the Go file-based skill content emission with the .NET behavior by always appending authoritative asset inventory blocks to loaded SKILL.md content. It ensures consumers (including LLM prompts) consistently see explicit <available_resources> and <available_scripts> blocks, including explicit empty self-closing blocks when no assets are present.

Changes:

• Append <available_resources> and <available_scripts> blocks to file-based skill content, emitting explicit empty blocks when lists are empty.
• Represent script schemas under <available_scripts> with nested <parameters_schema> elements (replacing the prior <script_schemas> shape).
• Update/extend tests to validate the new block shapes and the explicit empty-block behavior.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
agent/skills/fsskills/source.go	Appends new available-resources/scripts blocks to file skill content and implements the new XML block builders.
agent/skills/fsskills/source_script_test.go	Updates script-content expectations to the new <available_scripts> / <parameters_schema> structure and validates empty resources block presence.
agent/skills/fsskills/source_test.go	Adds coverage that file-based skill content includes explicit empty/non-empty <available_resources> blocks.
agent/skills/skills.go	Updates Script.ParametersSchema doc comment to reflect the new <available_scripts> emission location.
agent/skills/skills_test.go	Updates load_skill tool content expectation to include explicit empty available resources/scripts blocks.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Generated by Go API Consistency Review Agent for #394 · 409.4 AIC · ⌖ 13.2 AIC · ⊞ 27.1K

[michelle-clayton-work]

@copilot address the PR feedback

[Copilot]

@copilot address the PR feedback

Done — both parity gaps are now fixed in commit c07a2d3:

• buildAvailableResourcesBlock now emits description="..." on <resource> elements when resource.Description is non-empty, matching the .NET BuildAvailableResourcesBlock behavior.
• buildAvailableScriptsBlock now emits description="..." on <script> elements (both the self-closing and expanded forms) when script.Description is non-empty, matching the .NET BuildAvailableScriptsBlock behavior.
• Both attributes are XML-escaped via xmlEscapeAttr.
• Added white-box unit tests in source_block_builder_test.go covering the new conditional attribute logic and XML escaping for both resources and scripts.

[github-actions]

Cross-repo Parity Review ✅

This PR improves cross-language parity — no consistency issues found.

The previous Go implementation used a divergent <script_schemas> / <schema script="..."> format. This PR replaces it with <available_resources> and <available_scripts> blocks that precisely match the upstream .NET and Python implementations:

Behavior	.NET (AgentInlineSkillContentBuilder)	Python (_skills.py)	Go after this PR
Resource block name	<available_resources>	<available_resources>	<available_resources> ✅
Script block name	<available_scripts>	<available_scripts>	<available_scripts> ✅
Empty list	self-closing <available_resources />	self-closing <available_resources />	self-closing <available_resources /> ✅
Script with schema	<script name="..."><parameters_schema>...</parameters_schema></script>	same	same ✅
Separator	\n\n between original content and each block	\n\n	\n\n ✅
Attribute XML escaping	&, <, >, ", '	&, <, >, "	&, <, >, ", ' ✅
Content escaping (schema)	quotes preserved	quotes preserved	quotes preserved ✅

The Go changes to agent/skills/skills.go (comment update on ParametersSchema) and agent/skills/fsskills/source.go are semantically aligned with .NET's AgentFileSkill.GetContentAsync and Python's FileSkill.content property.

Generated by Go API Consistency Review Agent for #394 · 270 AIC · ⌖ 20.9 AIC · ⊞ 28.7K · ◷