modelcontextprotocol/mcpb
Watch
1
0
Fork
You've already forked mcpb
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix lint errors in types.ts and remove any casts

Browse source 
- Use import type for VERSIONED_MANIFEST_SCHEMAS and DEFAULT_MANIFEST_VERSION
- Remove unnecessary any type annotations from test file
This commit is contained in:
Joan Xie 2025-11-03 15:57:20 -08:00
commit 21c9710584
No known key found for this signature in database
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/types.ts
View file

@ -1,7 +1,8 @@
import type * as z from "zod";
import type { McpbManifestSchema as McpbManifestSchemaAny } from "./schemas/any.js";
import { VERSIONED_MANIFEST_SCHEMAS } from "./schemas/index.js";
import type { VERSIONED_MANIFEST_SCHEMAS } from "./schemas/index.js";
import type { DEFAULT_MANIFEST_VERSION } from "./shared/constants.js";
// Import schema types from the version matching DEFAULT_MANIFEST_VERSION
import type {
McpbManifestAuthorSchema,
@ -17,7 +18,6 @@ import type {
McpbUserConfigValuesSchema,
McpServerConfigSchema,
} from "./schemas/0.2.js";
import { DEFAULT_MANIFEST_VERSION } from "./shared/constants.js";
export type McpServerConfig = z.infer<typeof McpServerConfigSchema>;

4
test/schemas.test.ts
View file

@ -27,7 +27,7 @@ describe("McpbManifestSchema", () => {
expect(result.success).toBe(false);
if (!result.success) {
const errors = result.error.issues.map((issue: any) => issue.path.join("."));
const errors = result.error.issues.map((issue) => issue.path.join("."));
expect(errors).toContain("author.name");
expect(errors).toContain("author.email");
expect(errors).toContain("server.type");
@ -273,7 +273,7 @@ describe("McpbManifestSchema", () => {
const result = v0_3.McpbManifestSchema.safeParse(manifest);
expect(result.success).toBe(false);
if (!result.success) {
const messages = result.error.issues.map((issue: any) => issue.message);
const messages = result.error.issues.map((issue) => issue.message);
expect(messages.join(" ")).toContain("${locale}");
}
});