[PR #41] [MERGED] Reduce context usage: trim verbose examples and eliminate cross-file redundancy #42

New issue

Closed

opened 2026-03-15 11:42:45 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 11:42:45 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AvdLee/SwiftUI-Agent-Skill/pull/41
Author: @AvdLee
Created: 3/9/2026
Status: ✅ Merged
Merged: 3/10/2026
Merged by: @AvdLee

Base: main ← Head: optimize/reduce-context-usage

📝 Commits (9)

df71a3f Reduce context usage: trim verbose examples and eliminate cross-file redundancy
cb31607 Address Copilot review feedback
eac1c25 Revert unnecessary Copilot expansions and update copilot-instructions.md
6206cfc Address PR review feedback: split charts.md and fix remaining issues
ad19171 chore: sync README references [skip ci]
4eae6f7 Merge enhance-charts-reference into optimize/reduce-context-usage
e3f9ec4 Phase 2 context reduction: add TOCs and trim verbose examples
6efb473 Restore over-reduced content from PR review analysis
5c2289f Address Copilot review: fix foregroundStyle signature and coordinateSpace example

📊 Changes

18 files changed (+530 additions, -2055 deletions)

View changed files

📝 .github/copilot-instructions.md (+10 -0)
📝 README.md (+18 -24)
📝 swiftui-expert-skill/references/accessibility-patterns.md (+9 -0)
📝 swiftui-expert-skill/references/charts-accessibility.md (+27 -117)
📝 swiftui-expert-skill/references/charts.md (+90 -209)
📝 swiftui-expert-skill/references/image-optimization.md (+25 -117)
📝 swiftui-expert-skill/references/latest-apis.md (+44 -475)
📝 swiftui-expert-skill/references/layout-best-practices.md (+20 -80)
📝 swiftui-expert-skill/references/liquid-glass.md (+15 -0)
📝 swiftui-expert-skill/references/list-patterns.md (+11 -0)
📝 swiftui-expert-skill/references/macos-scenes.md (+40 -137)
📝 swiftui-expert-skill/references/macos-views.md (+43 -191)
📝 swiftui-expert-skill/references/macos-window-styling.md (+44 -149)
📝 swiftui-expert-skill/references/performance-patterns.md (+23 -93)
📝 swiftui-expert-skill/references/scroll-patterns.md (+8 -0)
📝 swiftui-expert-skill/references/sheet-navigation-patterns.md (+23 -180)
📝 swiftui-expert-skill/references/state-management.md (+53 -208)
📝 swiftui-expert-skill/references/view-structure.md (+27 -75)

📄 Description

Summary

Collapse simple deprecated-to-modern API renames into inline bullet lists instead of full code blocks
Remove duplicate code examples across reference files (Table/Person in macos-views.md, alert/dialog in sheet-navigation-patterns.md)
Shorten overly expanded examples (AXChartDescriptor, EditItemSheet, downsampling helpers, legacy ObservableObject patterns)
Remove composite "Complete Examples" section from charts.md (agents can compose from individual patterns)
Update README to clearly communicate the skill's breadth-to-context-efficiency ratio

Estimated Impact

File	Before	After	Saved
`latest-apis.md`	895	547	348
`sheet-navigation-patterns.md`	520	354	166
`charts.md`	930	798	132
`image-optimization.md`	295	172	123
`state-management.md`	572	452	120
`macos-views.md`	505	422	83
`performance-patterns.md`	473	397	76
`layout-best-practices.md`	326	255	71
`view-structure.md`	437	374	63
Total references	~7,998	~6,816	~1,182 (15%)

Zero information loss -- all patterns and guidance preserved, expressed more concisely. Files already lean (accessibility, animation-basics, animation-transitions, scroll-patterns) were left untouched.

Test plan

Verify all cross-references between files still resolve (e.g., list-patterns.md from macos-views.md, latest-apis.md from sheet-navigation-patterns.md)
Confirm no code examples are broken or incomplete after trimming
Verify README renders correctly on GitHub
Spot-check that trimmed examples still convey the intended pattern clearly

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AvdLee/SwiftUI-Agent-Skill/pull/41 **Author:** [@AvdLee](https://github.com/AvdLee) **Created:** 3/9/2026 **Status:** ✅ Merged **Merged:** 3/10/2026 **Merged by:** [@AvdLee](https://github.com/AvdLee) **Base:** `main` ← **Head:** `optimize/reduce-context-usage` --- ### 📝 Commits (9) - [`df71a3f`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/df71a3f939591441f7a0224c46d5949be5083f29) Reduce context usage: trim verbose examples and eliminate cross-file redundancy - [`cb31607`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/cb31607d7fa17670ea49891e8e530779ecd46dd7) Address Copilot review feedback - [`eac1c25`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/eac1c25902cd8efdac4b4e4b5d170d7c75cfcbc0) Revert unnecessary Copilot expansions and update copilot-instructions.md - [`6206cfc`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/6206cfc3d7fdea1c97be282514dae57a61fa9711) Address PR review feedback: split charts.md and fix remaining issues - [`ad19171`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/ad19171ecd007703be0bb208fdba9f8a44456768) chore: sync README references [skip ci] - [`4eae6f7`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/4eae6f712a0415601aa52c8fcb40d50cf313d206) Merge enhance-charts-reference into optimize/reduce-context-usage - [`e3f9ec4`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/e3f9ec4b2d03ccb8321c6a944112560b44259c3b) Phase 2 context reduction: add TOCs and trim verbose examples - [`6efb473`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/6efb4733485c88caf70cbf9ae494790fe4a47591) Restore over-reduced content from PR review analysis - [`5c2289f`](https://github.com/AvdLee/SwiftUI-Agent-Skill/commit/5c2289ffbc3aa5b02f25f73d84e178b08e8ea45c) Address Copilot review: fix foregroundStyle signature and coordinateSpace example ### 📊 Changes **18 files changed** (+530 additions, -2055 deletions) <details> <summary>View changed files</summary> 📝 `.github/copilot-instructions.md` (+10 -0) 📝 `README.md` (+18 -24) 📝 `swiftui-expert-skill/references/accessibility-patterns.md` (+9 -0) 📝 `swiftui-expert-skill/references/charts-accessibility.md` (+27 -117) 📝 `swiftui-expert-skill/references/charts.md` (+90 -209) 📝 `swiftui-expert-skill/references/image-optimization.md` (+25 -117) 📝 `swiftui-expert-skill/references/latest-apis.md` (+44 -475) 📝 `swiftui-expert-skill/references/layout-best-practices.md` (+20 -80) 📝 `swiftui-expert-skill/references/liquid-glass.md` (+15 -0) 📝 `swiftui-expert-skill/references/list-patterns.md` (+11 -0) 📝 `swiftui-expert-skill/references/macos-scenes.md` (+40 -137) 📝 `swiftui-expert-skill/references/macos-views.md` (+43 -191) 📝 `swiftui-expert-skill/references/macos-window-styling.md` (+44 -149) 📝 `swiftui-expert-skill/references/performance-patterns.md` (+23 -93) 📝 `swiftui-expert-skill/references/scroll-patterns.md` (+8 -0) 📝 `swiftui-expert-skill/references/sheet-navigation-patterns.md` (+23 -180) 📝 `swiftui-expert-skill/references/state-management.md` (+53 -208) 📝 `swiftui-expert-skill/references/view-structure.md` (+27 -75) </details> ### 📄 Description ## Summary - Collapse simple deprecated-to-modern API renames into inline bullet lists instead of full code blocks - Remove duplicate code examples across reference files (Table/Person in macos-views.md, alert/dialog in sheet-navigation-patterns.md) - Shorten overly expanded examples (AXChartDescriptor, EditItemSheet, downsampling helpers, legacy ObservableObject patterns) - Remove composite "Complete Examples" section from charts.md (agents can compose from individual patterns) - Update README to clearly communicate the skill's breadth-to-context-efficiency ratio ## Estimated Impact | File | Before | After | Saved | |------|--------|-------|-------| | `latest-apis.md` | 895 | 547 | 348 | | `sheet-navigation-patterns.md` | 520 | 354 | 166 | | `charts.md` | 930 | 798 | 132 | | `image-optimization.md` | 295 | 172 | 123 | | `state-management.md` | 572 | 452 | 120 | | `macos-views.md` | 505 | 422 | 83 | | `performance-patterns.md` | 473 | 397 | 76 | | `layout-best-practices.md` | 326 | 255 | 71 | | `view-structure.md` | 437 | 374 | 63 | | **Total references** | **~7,998** | **~6,816** | **~1,182 (15%)** | Zero information loss -- all patterns and guidance preserved, expressed more concisely. Files already lean (accessibility, animation-basics, animation-transitions, scroll-patterns) were left untouched. ## Test plan - [ ] Verify all cross-references between files still resolve (e.g., `list-patterns.md` from `macos-views.md`, `latest-apis.md` from `sheet-navigation-patterns.md`) - [ ] Confirm no code examples are broken or incomplete after trimming - [ ] Verify README renders correctly on GitHub - [ ] Spot-check that trimmed examples still convey the intended pattern clearly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-15 11:42:45 +03:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/SwiftUI-Agent-Skill#42

No description provided.

Rows
Columns