Commit 118717b
feat: GeneralUpdate Claude Code 技能套件(完整版) (#7)
* fix: resolve all 10 Copilot review comments
1. SemiUrsaClientView.axaml: Remove conv:StatusToVisibleConverter refs,
use CanExecute for button visibility
2. DownloadViewModels.cs: Sync VersionText on stats change,
clear SpeedText when not downloading
3. RealDownloadService.cs: Replace LaunchAsync() in CheckForUpdates
with direct HTTP /Upgrade/Verification call (true check-only mode)
4. FullIntegration.cs: Add missing using directives (Event, Configuration),
add hash length guard for truncation, fix speed unit duplication
5. reference.md: Fix framework version table (remove .NET Framework + .NET 8
conflation, use proper target framework monikers)
6. MinimalIntegration.cs: Fix System.Version comment accuracy
7. manifest.json: Remove JSON-invalid comment block, keep pure valid JSON
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: address Copilot review comments — API contracts, filenames, and API references
- Fix #1: RULES.md SetOption(Client) -> SetOption(AppType.Client)
- Fix #2: RULES.md manifest.json -> generalupdate.manifest.json
- Fix #3: backend-api.instructions.md add response wrapper { code, message, body }
- Fix #4: troubleshoot curl endpoint /api/update/check -> POST /Upgrade/Verification
- Fix #5: cursor/generalupdate-init.mdc manifest.json -> generalupdate.manifest.json
- Fix #6: clinerules/01-generalupdate-init.md manifest.json -> generalupdate.manifest.json
- Fix #7: troubleshoot LongPathSupport -> generic .NET long path guidance
- Fix #8: troubleshoot ZipSecurity -> entry path validation behavior
- Fix #9: troubleshoot SignalRRetryPolicy -> retry intent description
Co-Authored-By: Claude <noreply@anthropic.com>
* i18n: convert all Chinese comments and docs to English, make skills bilingual
All code comments, XML docs, console log strings, UI labels, and documentation
now use English (with bilingual status labels where helpful).
Converted files include:
- All 5 SKILL.md files (init/ui/strategy/advanced/troubleshoot)
- All 6 strategy example .cs files
- All 4 advanced template .cs files
- All 3 project scaffold files
- All 10+ UI template files (XAML/AXAML/CS)
- reference.md and troubleshooting reference
- All adapter files (.cursor/rules/, .clinerules/, .github/, RULES.md)
Skills now support both English and Chinese-speaking developers
through bilingual status labels in UI templates.
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: resolve all 6 Copilot review comments on PR #7
1. MinimalIntegration.cs: Fix System.Version comment accuracy
2. RealDownloadService.cs: Accept _updateUrl as-is (no /Upgrade/Verification append),
add tenantId, use TryGetProperty for response parsing, add productId param,
lowercase platform values
3. NamedPipeIPC.cs: Use Environment.ProcessId for unique pipe name,
enforce timeoutMs via linked CancellationTokenSource with TimeoutException
4. clinerules/05: Fix curl appType from 1 to 0, fix manifest path
---------
Co-authored-by: JusterZhu <juster.zhu@example.com>
Co-authored-by: Claude <noreply@anthropic.com>1 parent 231c1c2 commit 118717b
31 files changed
Lines changed: 806 additions & 977 deletions
File tree
- .claude/skills
- generalupdate-advanced/templates
- generalupdate-init
- project-scaffold
- templates
- generalupdate-strategy/examples
- generalupdate-troubleshoot
- generalupdate-ui
- templates
- .clinerules
- .cursor/rules
- .github/instructions
Lines changed: 17 additions & 38 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
17 | 11 | | |
18 | 12 | | |
19 | 13 | | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
25 | 19 | | |
26 | 20 | | |
27 | 21 | | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | 22 | | |
32 | 23 | | |
33 | | - | |
| 24 | + | |
34 | 25 | | |
35 | 26 | | |
36 | | - | |
37 | | - | |
38 | 27 | | |
39 | 28 | | |
40 | | - | |
41 | | - | |
42 | | - | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
43 | 32 | | |
44 | | - | |
45 | 33 | | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
| 34 | + | |
50 | 35 | | |
51 | 36 | | |
52 | | - | |
53 | 37 | | |
54 | 38 | | |
55 | | - | |
56 | 39 | | |
57 | | - | |
58 | 40 | | |
59 | | - | |
60 | 41 | | |
61 | | - | |
62 | | - | |
63 | | - | |
| 42 | + | |
64 | 43 | | |
65 | 44 | | |
66 | 45 | | |
Lines changed: 22 additions & 50 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
6 | 8 | | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
| 9 | + | |
11 | 10 | | |
12 | 11 | | |
13 | 12 | | |
14 | 13 | | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
| 14 | + | |
19 | 15 | | |
20 | 16 | | |
21 | | - | |
| 17 | + | |
22 | 18 | | |
23 | | - | |
| 19 | + | |
24 | 20 | | |
25 | | - | |
| 21 | + | |
26 | 22 | | |
27 | | - | |
| 23 | + | |
28 | 24 | | |
29 | 25 | | |
30 | 26 | | |
31 | | - | |
| 27 | + | |
32 | 28 | | |
33 | 29 | | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
| 30 | + | |
38 | 31 | | |
39 | 32 | | |
40 | | - | |
41 | | - | |
| 33 | + | |
42 | 34 | | |
43 | 35 | | |
44 | 36 | | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
| 37 | + | |
49 | 38 | | |
50 | 39 | | |
51 | | - | |
52 | | - | |
53 | | - | |
| 40 | + | |
54 | 41 | | |
55 | 42 | | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
| 43 | + | |
61 | 44 | | |
62 | 45 | | |
63 | 46 | | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
| 47 | + | |
68 | 48 | | |
69 | 49 | | |
70 | | - | |
71 | | - | |
72 | | - | |
73 | | - | |
| 50 | + | |
| 51 | + | |
74 | 52 | | |
75 | 53 | | |
76 | 54 | | |
77 | 55 | | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
| 56 | + | |
83 | 57 | | |
84 | 58 | | |
85 | | - | |
| 59 | + | |
86 | 60 | | |
87 | | - | |
| 61 | + | |
88 | 62 | | |
89 | 63 | | |
90 | 64 | | |
91 | 65 | | |
92 | | - | |
93 | 66 | | |
94 | | - | |
95 | 67 | | |
96 | 68 | | |
97 | | - | |
| 69 | + | |
98 | 70 | | |
99 | 71 | | |
Lines changed: 18 additions & 46 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
| 6 | + | |
6 | 7 | | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
| 8 | + | |
11 | 9 | | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | 10 | | |
17 | 11 | | |
18 | 12 | | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | 13 | | |
23 | 14 | | |
24 | | - | |
| 15 | + | |
25 | 16 | | |
26 | | - | |
| 17 | + | |
27 | 18 | | |
28 | 19 | | |
29 | | - | |
| 20 | + | |
30 | 21 | | |
31 | 22 | | |
32 | 23 | | |
33 | | - | |
| 24 | + | |
34 | 25 | | |
35 | 26 | | |
36 | | - | |
| 27 | + | |
37 | 28 | | |
38 | | - | |
39 | 29 | | |
40 | | - | |
41 | | - | |
| 30 | + | |
| 31 | + | |
42 | 32 | | |
43 | 33 | | |
44 | | - | |
45 | 34 | | |
46 | | - | |
47 | | - | |
| 35 | + | |
48 | 36 | | |
49 | 37 | | |
50 | | - | |
| 38 | + | |
51 | 39 | | |
52 | 40 | | |
53 | | - | |
| 41 | + | |
54 | 42 | | |
55 | 43 | | |
56 | 44 | | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | 45 | | |
61 | 46 | | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
| 47 | + | |
| 48 | + | |
68 | 49 | | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | | - | |
| 50 | + | |
73 | 51 | | |
74 | 52 | | |
75 | 53 | | |
| |||
79 | 57 | | |
80 | 58 | | |
81 | 59 | | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | | - | |
| 60 | + | |
86 | 61 | | |
87 | | - | |
| 62 | + | |
88 | 63 | | |
89 | 64 | | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | 65 | | |
94 | 66 | | |
95 | 67 | | |
| |||
0 commit comments