Commit 31889d1
committed
fix: correct Context type parameters in examples and tests
Context is Generic[LifespanContextT, RequestT], not Generic[ServerSessionT, ...].
The Context[ServerSession, None] pattern scattered across examples, tests, and
docs was passing ServerSession as LifespanContextT, which is semantically wrong
(the type checker only accepted it because LifespanContextT is unbounded).
Both TypeVars have PEP 696 defaults (dict[str, Any] and Any respectively), so
bare Context is the correct annotation when neither parameter needs to be
constrained. The one exception is the lifespan example, which correctly uses
Context[AppContext] to type the lifespan context.
This also removes the now-unused ServerSession imports from the affected files.1 parent 7ba41dc commit 31889d1
File tree
13 files changed
+66
-88
lines changed- examples
- servers/everything-server/mcp_everything_server
- snippets/servers
- tests/server
- mcpserver
13 files changed
+66
-88
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
229 | 229 | | |
230 | 230 | | |
231 | 231 | | |
232 | | - | |
233 | 232 | | |
234 | 233 | | |
235 | 234 | | |
| |||
275 | 274 | | |
276 | 275 | | |
277 | 276 | | |
278 | | - | |
| 277 | + | |
279 | 278 | | |
280 | 279 | | |
281 | 280 | | |
| |||
347 | 346 | | |
348 | 347 | | |
349 | 348 | | |
350 | | - | |
351 | 349 | | |
352 | 350 | | |
353 | 351 | | |
354 | 352 | | |
355 | 353 | | |
356 | | - | |
| 354 | + | |
357 | 355 | | |
358 | 356 | | |
359 | 357 | | |
| |||
695 | 693 | | |
696 | 694 | | |
697 | 695 | | |
698 | | - | |
699 | 696 | | |
700 | 697 | | |
701 | 698 | | |
702 | 699 | | |
703 | 700 | | |
704 | | - | |
| 701 | + | |
705 | 702 | | |
706 | 703 | | |
707 | 704 | | |
| |||
828 | 825 | | |
829 | 826 | | |
830 | 827 | | |
831 | | - | |
832 | 828 | | |
833 | 829 | | |
834 | 830 | | |
| |||
846 | 842 | | |
847 | 843 | | |
848 | 844 | | |
849 | | - | |
| 845 | + | |
850 | 846 | | |
851 | 847 | | |
852 | 848 | | |
| |||
870 | 866 | | |
871 | 867 | | |
872 | 868 | | |
873 | | - | |
| 869 | + | |
874 | 870 | | |
875 | 871 | | |
876 | 872 | | |
| |||
894 | 890 | | |
895 | 891 | | |
896 | 892 | | |
897 | | - | |
| 893 | + | |
898 | 894 | | |
899 | 895 | | |
900 | 896 | | |
| |||
935 | 931 | | |
936 | 932 | | |
937 | 933 | | |
938 | | - | |
939 | 934 | | |
940 | 935 | | |
941 | 936 | | |
942 | 937 | | |
943 | 938 | | |
944 | 939 | | |
945 | | - | |
| 940 | + | |
946 | 941 | | |
947 | 942 | | |
948 | 943 | | |
| |||
972 | 967 | | |
973 | 968 | | |
974 | 969 | | |
975 | | - | |
976 | 970 | | |
977 | 971 | | |
978 | 972 | | |
979 | 973 | | |
980 | 974 | | |
981 | | - | |
| 975 | + | |
982 | 976 | | |
983 | 977 | | |
984 | 978 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
346 | 346 | | |
347 | 347 | | |
348 | 348 | | |
349 | | - | |
350 | 349 | | |
351 | 350 | | |
352 | 351 | | |
353 | 352 | | |
354 | 353 | | |
355 | | - | |
| 354 | + | |
356 | 355 | | |
357 | 356 | | |
358 | 357 | | |
| |||
694 | 693 | | |
695 | 694 | | |
696 | 695 | | |
697 | | - | |
698 | 696 | | |
699 | 697 | | |
700 | 698 | | |
701 | 699 | | |
702 | 700 | | |
703 | | - | |
| 701 | + | |
704 | 702 | | |
705 | 703 | | |
706 | 704 | | |
| |||
826 | 824 | | |
827 | 825 | | |
828 | 826 | | |
829 | | - | |
830 | 827 | | |
831 | 828 | | |
832 | 829 | | |
| |||
844 | 841 | | |
845 | 842 | | |
846 | 843 | | |
847 | | - | |
| 844 | + | |
848 | 845 | | |
849 | 846 | | |
850 | 847 | | |
| |||
868 | 865 | | |
869 | 866 | | |
870 | 867 | | |
871 | | - | |
| 868 | + | |
872 | 869 | | |
873 | 870 | | |
874 | 871 | | |
| |||
892 | 889 | | |
893 | 890 | | |
894 | 891 | | |
895 | | - | |
| 892 | + | |
896 | 893 | | |
897 | 894 | | |
898 | 895 | | |
| |||
933 | 930 | | |
934 | 931 | | |
935 | 932 | | |
936 | | - | |
937 | 933 | | |
938 | 934 | | |
939 | 935 | | |
940 | 936 | | |
941 | 937 | | |
942 | 938 | | |
943 | | - | |
| 939 | + | |
944 | 940 | | |
945 | 941 | | |
946 | 942 | | |
| |||
970 | 966 | | |
971 | 967 | | |
972 | 968 | | |
973 | | - | |
974 | 969 | | |
975 | 970 | | |
976 | 971 | | |
977 | 972 | | |
978 | 973 | | |
979 | | - | |
| 974 | + | |
980 | 975 | | |
981 | 976 | | |
982 | 977 | | |
| |||
Lines changed: 7 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | 16 | | |
18 | 17 | | |
19 | 18 | | |
| |||
142 | 141 | | |
143 | 142 | | |
144 | 143 | | |
145 | | - | |
| 144 | + | |
146 | 145 | | |
147 | 146 | | |
148 | 147 | | |
| |||
155 | 154 | | |
156 | 155 | | |
157 | 156 | | |
158 | | - | |
| 157 | + | |
159 | 158 | | |
160 | 159 | | |
161 | 160 | | |
| |||
173 | 172 | | |
174 | 173 | | |
175 | 174 | | |
176 | | - | |
| 175 | + | |
177 | 176 | | |
178 | 177 | | |
179 | 178 | | |
| |||
198 | 197 | | |
199 | 198 | | |
200 | 199 | | |
201 | | - | |
| 200 | + | |
202 | 201 | | |
203 | 202 | | |
204 | 203 | | |
| |||
230 | 229 | | |
231 | 230 | | |
232 | 231 | | |
233 | | - | |
| 232 | + | |
234 | 233 | | |
235 | 234 | | |
236 | 235 | | |
| |||
289 | 288 | | |
290 | 289 | | |
291 | 290 | | |
292 | | - | |
| 291 | + | |
293 | 292 | | |
294 | 293 | | |
295 | 294 | | |
| |||
313 | 312 | | |
314 | 313 | | |
315 | 314 | | |
316 | | - | |
| 315 | + | |
317 | 316 | | |
318 | 317 | | |
319 | 318 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | 13 | | |
15 | 14 | | |
16 | 15 | | |
| |||
28 | 27 | | |
29 | 28 | | |
30 | 29 | | |
31 | | - | |
| 30 | + | |
32 | 31 | | |
33 | 32 | | |
34 | 33 | | |
| |||
52 | 51 | | |
53 | 52 | | |
54 | 53 | | |
55 | | - | |
| 54 | + | |
56 | 55 | | |
57 | 56 | | |
58 | 57 | | |
| |||
76 | 75 | | |
77 | 76 | | |
78 | 77 | | |
79 | | - | |
| 78 | + | |
80 | 79 | | |
81 | 80 | | |
82 | 81 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | 2 | | |
4 | 3 | | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
8 | | - | |
| 7 | + | |
9 | 8 | | |
10 | 9 | | |
11 | 10 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | 2 | | |
4 | 3 | | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
8 | 7 | | |
9 | | - | |
| 8 | + | |
10 | 9 | | |
11 | 10 | | |
12 | 11 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | 2 | | |
4 | 3 | | |
5 | 4 | | |
6 | 5 | | |
7 | 6 | | |
8 | | - | |
| 7 | + | |
9 | 8 | | |
10 | 9 | | |
11 | 10 | | |
| |||
0 commit comments