Skip to content

[3/8] Clarify evaluation, SDK, and performance context - #14964

Draft
JanProvaznik wants to merge 1 commit into
janprovaznik/context-02-contractsfrom
janprovaznik/context-03-evaluation
Draft

[3/8] Clarify evaluation, SDK, and performance context#14964
JanProvaznik wants to merge 1 commit into
janprovaznik/context-02-contractsfrom
janprovaznik/context-03-evaluation

Conversation

@JanProvaznik

@JanProvaznik JanProvaznik commented Sep 7, 2026

Copy link
Copy Markdown
Member

Part 3 of 8

Correct property/import ordering, globbing and target semantics, and replace blanket optimization rules with contract-specific, measured guidance.

Depends on #14963. Next: #14965. Full index: #14961.

This PR is based on janprovaznik/context-02-contracts. Files changed shows only this chapter. Review bottom-up; after the preceding chapter lands, retarget this PR to main before merging. Do not merge later chapters into temporary stack-base branches.

Scope and evidence

Guidance is checked against the referenced source/configuration, and this chapter introduces no new broken local references. No product runtime behavior is claimed to have been re-executed by a documentation change.

Agentic-workflow changes remain separate. This stack does not include personal plugin overrides, plugin auto-activation changes, or SDK/feed version changes. The public descriptions use public repository evidence, not internal discussions or proprietary source.

Correct property/import ordering, globbing and target semantics, and replace blanket optimization rules with contract-specific, measured guidance.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 5e665c32-8412-4c24-bf55-55680480d04d
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

🔍 Skill Validator Results

⚠️ Warnings or advisories found

Scope Checked
Skills 2
Agents 0
Total 2
Severity Count
--- ---:
❌ Errors 0
⚠️ Warnings 2
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ Found 2 skill(s)
ℹ️ [optimizing-msbuild-performance] 📊 optimizing-msbuild-performance: 445 BPE tokens [chars/4: 597] (detailed ✓), 4 sections, 0 code blocks
ℹ️ [optimizing-msbuild-performance] ⚠ No code blocks — agents perform better with concrete snippets and commands.
ℹ️ [integrating-sdk-and-msbuild] 📊 integrating-sdk-and-msbuild: 509 BPE tokens [chars/4: 671] (detailed ✓), 4 sections, 0 code blocks
ℹ️ [integrating-sdk-and-msbuild] ⚠ No code blocks — agents perform better with concrete snippets and commands.
ℹ️ ✅ All checks passed (2 skill(s))
Full validator output ```text Found 2 skill(s) [optimizing-msbuild-performance] 📊 optimizing-msbuild-performance: 445 BPE tokens [chars/4: 597] (detailed ✓), 4 sections, 0 code blocks [optimizing-msbuild-performance] ⚠ No code blocks — agents perform better with concrete snippets and commands. [integrating-sdk-and-msbuild] 📊 integrating-sdk-and-msbuild: 509 BPE tokens [chars/4: 671] (detailed ✓), 4 sections, 0 code blocks [integrating-sdk-and-msbuild] ⚠ No code blocks — agents perform better with concrete snippets and commands. ✅ All checks passed (2 skill(s)) ```

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.

1 participant