Skip to content

Commit af9e2d3

Browse files
committed
fix: address review feedback
1 parent cd5c324 commit af9e2d3

2 files changed

Lines changed: 6 additions & 49 deletions

File tree

schemas.ts

Lines changed: 4 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -566,56 +566,11 @@ export const GitLabNamespaceSchema = z.object({
566566
kind: z.enum(["user", "group"]),
567567
full_path: z.string(),
568568
parent_id: z.coerce.string().nullable(),
569-
avatar_url: z.string().nullable(),
569+
avatar_url: z.string().nullable().optional(),
570570
web_url: z.string(),
571-
members_count_with_descendants: z.coerce.number().optional(),
572-
billable_members_count: z.coerce.number().optional(),
573-
max_seats_used: z.coerce.number().optional(),
574-
seats_in_use: z.coerce.number().optional(),
575-
plan: z.string().optional(),
576-
end_date: z.string().nullable().optional(),
577-
trial_ends_on: z.string().nullable().optional(),
578-
trial: z.coerce.boolean().optional(),
579-
root_repository_size: z.coerce.number().optional(),
580-
projects_count: z.coerce.number().optional(),
581-
});
582-
583-
export const GitLabNamespaceExistsResponseSchema = z.object({
584-
exists: z.coerce.boolean(),
585-
suggests: z.array(z.string()).optional(),
586-
});
587-
588-
// Repository related schemas
589-
export const GitLabOwnerSchema = z.object({
590-
username: z.string(), // Changed from login to match GitLab API
591-
id: z.coerce.string(),
592-
avatar_url: z.string().nullable(),
593-
web_url: z.string(), // Changed from html_url to match GitLab API
594-
name: z.string(), // Added as GitLab includes full name
595-
state: z.string(), // Added as GitLab includes user state
596-
});
597-
598-
export const GitLabRepositorySchema = z.object({
599-
id: z.coerce.string(),
600-
name: z.string(),
601-
path_with_namespace: z.string(),
602-
visibility: z.string().optional(),
603-
owner: GitLabOwnerSchema.optional(),
604-
web_url: z.string().optional(),
605-
description: z.string().nullable(),
606-
fork: z.coerce.boolean().optional(),
607-
ssh_url_to_repo: z.string().optional(),
608-
http_url_to_repo: z.string().optional(),
609-
created_at: z.string().optional(),
610-
last_activity_at: z.string().optional(),
611-
default_branch: z.string().nullable().optional(),
612-
namespace: z
613-
.object({
614-
id: z.coerce.string(),
615-
name: z.string(),
616-
path: z.string(),
617-
kind: z.string(),
618-
full_path: z.string(),
571+
request_access_enabled: z.boolean().optional(),
572+
full_name: z.string(),
573+
full_path: z.string(),
619574
avatar_url: z.string().nullable().optional(),
620575
web_url: z.string().optional(),
621576
})

tools/registry.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1163,6 +1163,7 @@ export const readOnlyTools = new Set([
11631163
"list_webhooks",
11641164
"list_webhook_events",
11651165
"get_webhook_event",
1166+
"health_check",
11661167
]);
11671168

11681169
// Define which tools are destructive (data loss potential)
@@ -1380,6 +1381,7 @@ export const TOOLSET_DEFINITIONS: readonly ToolsetDefinition[] = [
13801381
"verify_namespace",
13811382
"list_group_projects",
13821383
"list_group_iterations",
1384+
"health_check",
13831385
]),
13841386
},
13851387
{

0 commit comments

Comments
 (0)