Skip to content

Improved sample project #93

Merged
ArcadeMode merged 18 commits intomasterfrom
sample-tweaks
Mar 3, 2026
Merged

Improved sample project #93
ArcadeMode merged 18 commits intomasterfrom
sample-tweaks

Conversation

@ArcadeMode
Copy link
Owner

@ArcadeMode ArcadeMode commented Mar 3, 2026

#86

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Test Results

    9 files     73 suites   33s ⏱️
  695 tests   691 ✅  4 💤 0 ❌
1 986 runs  1 961 ✅ 25 💤 0 ❌

Results for commit bbb0165.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Mar 3, 2026


BenchmarkDotNet v0.15.8, Linux Ubuntu 24.04.3 LTS (Noble Numbat)
AMD EPYC 7763 2.76GHz, 1 CPU, 4 logical and 2 physical cores
.NET SDK 10.0.103
  [Host]     : .NET 10.0.3 (10.0.3, 10.0.326.7603), X64 RyuJIT x86-64-v3
  Job-MLTFGY : .NET 10.0.3 (10.0.3, 10.0.326.7603), X64 RyuJIT x86-64-v3

InvocationCount=1  IterationCount=1  LaunchCount=10  
RunStrategy=ColdStart  UnrollFactor=1  WarmupCount=0  

Method Compilation ClassCount Mean Error StdDev Min Max
Generate AOT 0 14.00 ms 0.465 ms 0.307 ms 13.27 ms 14.31 ms
Generate AOT 1 31.33 ms 1.157 ms 0.765 ms 30.12 ms 32.61 ms
Generate AOT 10 31.08 ms 0.745 ms 0.493 ms 30.16 ms 31.77 ms
Generate AOT 25 43.00 ms 1.166 ms 0.771 ms 41.81 ms 44.55 ms
Generate AOT 50 55.85 ms 2.397 ms 1.585 ms 53.85 ms 59.68 ms
Generate AOT 100 87.91 ms 1.766 ms 1.168 ms 86.35 ms 89.65 ms
Generate AOT 200 91.27 ms 0.560 ms 0.370 ms 90.77 ms 91.80 ms
Generate JIT 0 40.16 ms 0.320 ms 0.211 ms 39.82 ms 40.51 ms
Generate JIT 1 797.53 ms 3.630 ms 2.401 ms 793.77 ms 801.16 ms
Generate JIT 10 797.51 ms 6.305 ms 4.171 ms 793.03 ms 803.98 ms
Generate JIT 25 840.45 ms 7.873 ms 5.208 ms 834.43 ms 848.81 ms
Generate JIT 50 868.02 ms 3.618 ms 2.393 ms 863.53 ms 871.75 ms
Generate JIT 100 965.17 ms 6.677 ms 4.417 ms 958.91 ms 970.07 ms
Generate JIT 200 951.29 ms 7.597 ms 5.025 ms 945.74 ms 961.41 ms

@ArcadeMode
Copy link
Owner Author

@copilot take the following steps to make changes:

Step 1 change the naming of projects in the sample directory.

  • TypeShim.Sample into Client.Library
  • TypeShim.Sample.Server into Server

Step 2 is to remove the typeshim.sample.client directory and pull its contents up into the sample directory. Also remove the .esproj file and also remove from the slnx.

Step 3 is renaming @typeshim to @client both on disk and in the npm projects/package.jsons and code

Step 4 is merging npm project @client/people-ui into @client/app. Put the components from people-ui into a directory people in the app project. Then remove unnecessary files left over from the merge.

Copy link
Contributor

Copilot AI commented Mar 3, 2026

@ArcadeMode I've opened a new pull request, #94, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 5 commits March 3, 2026 02:39
…lient, merge people-ui into app (#94)

Cleans up the sample project layout — flatter structure, consistent
naming, fewer npm packages.

## Changes

- **Project renames**
  - `TypeShim.Sample` → `Client.Library` (dir, csproj, C# namespaces)
  - `TypeShim.Sample.Server` → `Server` (dir, csproj, C# namespaces)
  - `Sample.slnx` updated; `Server.csproj` project reference updated

- **Flatten client directory**
- `TypeShim.Sample.Client/` removed; contents (`package.json`,
workspaces, `.gitignore`, etc.) lifted into `sample/` root
  - `.esproj` / `.esproj.user` deleted; removed from `Sample.slnx`
  - `Client.Library.csproj` `<OutputPath>` updated to new path

- **`@typeshim` → `@client`**
- Directory, all `package.json` names, TypeScript imports, string
literals, `.gitignore` glob pattern

- **Merge `@client/people-ui` into `@client/app`**
  - Components moved to `@client/app/src/people/`
  - `@client/app` no longer depends on `@client/people-ui`
- `build:people` script removed; `package-lock.json` and root
`tsconfig.json` references cleaned up
  - `@client/people-ui` package deleted

<!-- START COPILOT CODING AGENT TIPS -->
---

✨ Let Copilot coding agent [set things up for
you](https://github.com/ArcadeMode/TypeShim/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot)
— coding agent works faster and does higher quality work when set up for
your repo.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ArcadeMode <5969155+ArcadeMode@users.noreply.github.com>
@ArcadeMode ArcadeMode merged commit d57d204 into master Mar 3, 2026
13 checks passed
@ArcadeMode ArcadeMode deleted the sample-tweaks branch March 3, 2026 20:21
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