pi-revit 0.4.0 → 0.5.0

package/src/Revit/Tools/GetModelCoordinates.cs

@@ -1,49 +1,56 @@
-using System.Text.Json;
-using Autodesk.Revit.DB;
-
-namespace RevitBridge.Tools;
-
-internal sealed class GetModelCoordinates : ITool
-{
-    public string Name => "get_model_coordinates";
-    public string Label => "Get Model Coordinates";
-    public string Tier => "advanced";
-    public string Description => "Read project/survey base points, active project location and site coordinates. Optionally report shared coordinates for up to 100 points expressed along document internal axes. Required unit applies to input points and every returned length; angles and latitude/longitude are degrees. The active location's Revit GetProjectPosition defines the shared mapping. List project locations with paging. Does not infer a GIS coordinate reference system or modify/acquire coordinates.";
-    public object ParametersSchema => new
-    {
-        type = "object", properties = new
-        {
-            unit = ModelEditInputs.LengthUnitSchema,
-            points = new { type = "array", maxItems = 100, items = ModelEditInputs.VectorSchema("Point along document internal axes, in unit.") },
-            offset = new { type = "integer", minimum = 0 }, limit = new { type = "integer", minimum = 1, maximum = 100 },
-        }, required = new[] { "unit" },
-    };
-    public object Execute(JsonElement args, ToolContext context)
-    {
-        var doc = context.Document ?? throw new NoActiveDocumentException();
-        if (doc.IsFamilyDocument) throw new ArgumentException("Project coordinates require a project document.");
-        double scale = ModelEditInputs.LengthScale(args);
-        var active = doc.ActiveProjectLocation;
-        object Position(ProjectPosition position) => new { east_west = position.EastWest / scale, north_south = position.NorthSouth / scale, elevation = position.Elevation / scale, angle_degrees = position.Angle * 180 / Math.PI };
-        object? Base(BasePoint? point) => point == null ? null : new { id = point.Id.Value, internal_position = SpatialBounds.Coordinates(point.Position, scale), shared_position = SpatialBounds.Coordinates(point.SharedPosition, scale) };
-        var points = new List<object>();
-        if (args.TryGetProperty("points", out var values))
-        {
-            if (values.ValueKind != JsonValueKind.Array || values.GetArrayLength() > 100) throw new ArgumentException("points must be an array of at most 100 XYZ points.");
-            foreach (var value in values.EnumerateArray())
-            {
-                var wrapper = JsonSerializer.SerializeToElement(new { point = value });
-                var point = ModelEditInputs.Vector(wrapper, "point").Multiply(scale);
-                points.Add(new { internal_position = SpatialBounds.Coordinates(point, scale), shared_position = Position(active.GetProjectPosition(point)) });
-            }
-        }
-        var locations = doc.ProjectLocations.Cast<ProjectLocation>().OrderBy(x => x.Id.Value).ToList();
-        int offset = Math.Max(0, JsonArgs.GetInt(args, "offset", 0)), limit = Math.Clamp(JsonArgs.GetInt(args, "limit", 50), 1, 100);
-        var page = locations.Skip(offset).Take(limit).Select(x => new { id = x.Id.Value, name = x.Name, active = x.Id == active.Id, internal_origin_in_shared_coordinates = Position(x.GetProjectPosition(XYZ.Zero)) }).ToArray();
-        var site = doc.SiteLocation;
-        return new { unit = JsonArgs.GetString(args, "unit"), angle_unit = "degrees", point_input_coordinates = "document_internal", active_project_location = new { id = active.Id.Value, name = active.Name },
-            project_base_point = Base(BasePoint.GetProjectBasePoint(doc)), survey_point = Base(BasePoint.GetSurveyPoint(doc)),
-            site = new { name = site.Name, latitude_degrees = site.Latitude * 180 / Math.PI, longitude_degrees = site.Longitude * 180 / Math.PI, time_zone = site.TimeZone },
-            total_locations = locations.Count, offset, next_offset = offset + page.Length < locations.Count ? (int?)(offset + page.Length) : null, locations = page, points };
-    }
-}
+using System.Text.Json;
+using Autodesk.Revit.DB;
+
+namespace RevitBridge.Tools;
+
+internal sealed class GetModelCoordinates : ITool
+{
+    public string Name => "get_model_coordinates";
+    public IReadOnlyList<string> DocumentKinds => DocumentKind.ProjectOnly;
+    public IReadOnlyList<string> Keywords => new[] { "survey point", "project base point", "true north", "project north", "shared coordinates", "site location", "latitude", "longitude", "georeference", "elevation" };
+    public IReadOnlyList<ToolLimit> Limits => new[]
+    {
+        new ToolLimit("Identifying the datum, projection or EPSG code", "user", "Confirm the coordinate reference system with the surveyor or project standard"),
+        new ToolLimit("Changing coordinates or acquiring shared coordinates", "api", "ProjectLocation.SetProjectPosition; Document.AcquireCoordinates"),
+    };
+    public string Label => "Get Model Coordinates";
+    public string Tier => "advanced";
+    public string Description => "Read project/survey base points, active project location and site coordinates. Optionally report shared coordinates for up to 100 points expressed along document internal axes. Required unit applies to input points and every returned length; angles and latitude/longitude are degrees. The active location's Revit GetProjectPosition defines the shared mapping. List project locations with paging. Does not infer a GIS coordinate reference system or modify/acquire coordinates.";
+    public object ParametersSchema => new
+    {
+        type = "object", properties = new
+        {
+            unit = ModelEditInputs.LengthUnitSchema,
+            points = new { type = "array", maxItems = 100, items = ModelEditInputs.VectorSchema("Point along document internal axes, in unit.") },
+            offset = new { type = "integer", minimum = 0 }, limit = new { type = "integer", minimum = 1, maximum = 100 },
+        }, required = new[] { "unit" },
+    };
+    public object Execute(JsonElement args, ToolContext context)
+    {
+        var doc = context.Document ?? throw new NoActiveDocumentException();
+        if (doc.IsFamilyDocument) throw new ArgumentException("Project coordinates require a project document.");
+        double scale = ModelEditInputs.LengthScale(args);
+        var active = doc.ActiveProjectLocation;
+        object Position(ProjectPosition position) => new { east_west = position.EastWest / scale, north_south = position.NorthSouth / scale, elevation = position.Elevation / scale, angle_degrees = position.Angle * 180 / Math.PI };
+        object? Base(BasePoint? point) => point == null ? null : new { id = point.Id.Value, internal_position = SpatialBounds.Coordinates(point.Position, scale), shared_position = SpatialBounds.Coordinates(point.SharedPosition, scale) };
+        var points = new List<object>();
+        if (args.TryGetProperty("points", out var values))
+        {
+            if (values.ValueKind != JsonValueKind.Array || values.GetArrayLength() > 100) throw new ArgumentException("points must be an array of at most 100 XYZ points.");
+            foreach (var value in values.EnumerateArray())
+            {
+                var wrapper = JsonSerializer.SerializeToElement(new { point = value });
+                var point = ModelEditInputs.Vector(wrapper, "point").Multiply(scale);
+                points.Add(new { internal_position = SpatialBounds.Coordinates(point, scale), shared_position = Position(active.GetProjectPosition(point)) });
+            }
+        }
+        var locations = doc.ProjectLocations.Cast<ProjectLocation>().OrderBy(x => x.Id.Value).ToList();
+        int offset = Math.Max(0, JsonArgs.GetInt(args, "offset", 0)), limit = Math.Clamp(JsonArgs.GetInt(args, "limit", 50), 1, 100);
+        var page = locations.Skip(offset).Take(limit).Select(x => new { id = x.Id.Value, name = x.Name, active = x.Id == active.Id, internal_origin_in_shared_coordinates = Position(x.GetProjectPosition(XYZ.Zero)) }).ToArray();
+        var site = doc.SiteLocation;
+        return new { unit = JsonArgs.GetString(args, "unit"), angle_unit = "degrees", point_input_coordinates = "document_internal", active_project_location = new { id = active.Id.Value, name = active.Name },
+            project_base_point = Base(BasePoint.GetProjectBasePoint(doc)), survey_point = Base(BasePoint.GetSurveyPoint(doc)),
+            site = new { name = site.Name, latitude_degrees = site.Latitude * 180 / Math.PI, longitude_degrees = site.Longitude * 180 / Math.PI, time_zone = site.TimeZone },
+            total_locations = locations.Count, offset, next_offset = offset + page.Length < locations.Count ? (int?)(offset + page.Length) : null, locations = page, points };
+    }
+}

package/src/Revit/Tools/GetModelHealth.cs

@@ -16,6 +16,13 @@ namespace RevitBridge.Tools
         private const int MaxElementsPerGroup = 20;
 
         public string Name => "get_model_health";
+        public IReadOnlyList<string> DocumentKinds => DocumentKind.Both;
+        public IReadOnlyList<string> Keywords => new[] { "warnings", "errors", "audit", "model quality", "health check", "worksets", "phases", "design options", "in-place families" };
+        public IReadOnlyList<ToolLimit> Limits => new[]
+        {
+            new ToolLimit("Resolving warnings (report only)", "api", "Document.GetWarnings; resolve with dedicated edit tools or custom code within scope"),
+            new ToolLimit("Geometry, performance or standards-compliance audits", "api", "Custom inspection; verify members with search_api_docs"),
+        };
         public string Label => "Get Model Health";
         public string Description => "Audit the open Revit model: every document warning grouped by description (description, severity, occurrence count, failing element ids+names — the top 100 groups ordered by count, up to 20 elements each), plus structure counts: worksharing status with user worksets, phases, design options, in-place families, and the total element count. Read-only; use it to review model quality, e.g. before and after bulk edits.";
         public string Tier => "advanced";

package/src/Revit/Tools/GetModelOverview.cs

@@ -1,160 +1,187 @@
-using System.Text.Json;
-using Autodesk.Revit.DB;
-
-namespace RevitBridge.Tools
-{
-    internal sealed class GetModelOverview : ITool
-    {
-        public string Name => "get_model_overview";
-        public string Label => "Get Model Overview";
-        public string Description => "One-call orientation for the open Revit model: project metadata (title, name, number, client, address, file path, Revit version, display units), levels with elevations, grids, element counts for the major categories, and totals (elements, views, sheets). Use this first when starting work on a model.";
-        public object ParametersSchema => new { type = "object", properties = new { }, required = Array.Empty<string>() };
-        public string? PromptSnippet => "Get project info, units, levels, grids, and category counts for the open Revit model.";
-        public IReadOnlyList<string>? PromptGuidelines => new[]
-        {
-            "Call get_model_overview first when starting work on a Revit model; it covers project info, units, levels, grids, and per-category counts in one call.",
-        };
-
-        private static readonly IReadOnlyDictionary<string, BuiltInCategory> MajorCategories = new Dictionary<string, BuiltInCategory>
-        {
-            ["Walls"] = BuiltInCategory.OST_Walls,
-            ["Doors"] = BuiltInCategory.OST_Doors,
-            ["Windows"] = BuiltInCategory.OST_Windows,
-            ["Floors"] = BuiltInCategory.OST_Floors,
-            ["Rooms"] = BuiltInCategory.OST_Rooms,
-            ["Areas"] = BuiltInCategory.OST_Areas,
-            ["Ceilings"] = BuiltInCategory.OST_Ceilings,
-            ["Roofs"] = BuiltInCategory.OST_Roofs,
-            ["Columns"] = BuiltInCategory.OST_Columns,
-            ["Structural Columns"] = BuiltInCategory.OST_StructuralColumns,
-            ["Structural Framing"] = BuiltInCategory.OST_StructuralFraming,
-            ["Stairs"] = BuiltInCategory.OST_Stairs,
-            ["Furniture"] = BuiltInCategory.OST_Furniture,
-            ["Generic Models"] = BuiltInCategory.OST_GenericModel,
-            ["Pipes"] = BuiltInCategory.OST_PipeCurves,
-            ["Ducts"] = BuiltInCategory.OST_DuctCurves,
-        };
-
-        public object? Execute(JsonElement args, ToolContext context)
-        {
-            var doc = context.Document ?? throw new NoActiveDocumentException();
-            var info = doc.ProjectInformation;
-            var units = doc.GetUnits();
-
-            string system = doc.DisplayUnitSystem == DisplayUnit.IMPERIAL ? "imperial" : "metric";
-            string? lengthUnit = DisplayUnitName(units, SpecTypeId.Length);
-            var project = new Dictionary<string, object?>
-            {
+using System.Text.Json;
+using Autodesk.Revit.DB;
+
+namespace RevitBridge.Tools
+{
+    internal sealed class GetModelOverview : ITool
+    {
+        public string Name => "get_model_overview";
+        public IReadOnlyList<string> DocumentKinds => DocumentKind.Both;
+        public IReadOnlyList<string> Keywords => new[] { "project info", "model info", "units", "levels", "grids", "category counts", "summary", "document id", "identity", "document kind", "project or family", "family document" };
+        public IReadOnlyList<ToolLimit> Limits => new[]
+        {
+            new ToolLimit("Switching the active document or Revit session", "tool", "manage_revit_instances"),
+        };
+        public string Label => "Get Model Overview";
+        public string Description => "One-call orientation for the open Revit model: project metadata (title, name, number, client, address, file path, Revit version, display units), levels with elevations, grids, element counts for the major categories, and totals (elements, views, sheets). Use this first when starting work on a model.";
+        public object ParametersSchema => new { type = "object", properties = new { }, required = Array.Empty<string>() };
+        public string? PromptSnippet => "Get project info, units, levels, grids, and category counts for the open Revit model.";
+        public IReadOnlyList<string>? PromptGuidelines => new[]
+        {
+            "Call get_model_overview first when starting work on a Revit model; it covers project info, units, levels, grids, and per-category counts in one call.",
+        };
+
+        private static readonly IReadOnlyDictionary<string, BuiltInCategory> MajorCategories = new Dictionary<string, BuiltInCategory>
+        {
+            ["Walls"] = BuiltInCategory.OST_Walls,
+            ["Doors"] = BuiltInCategory.OST_Doors,
+            ["Windows"] = BuiltInCategory.OST_Windows,
+            ["Floors"] = BuiltInCategory.OST_Floors,
+            ["Rooms"] = BuiltInCategory.OST_Rooms,
+            ["Areas"] = BuiltInCategory.OST_Areas,
+            ["Ceilings"] = BuiltInCategory.OST_Ceilings,
+            ["Roofs"] = BuiltInCategory.OST_Roofs,
+            ["Columns"] = BuiltInCategory.OST_Columns,
+            ["Structural Columns"] = BuiltInCategory.OST_StructuralColumns,
+            ["Structural Framing"] = BuiltInCategory.OST_StructuralFraming,
+            ["Stairs"] = BuiltInCategory.OST_Stairs,
+            ["Furniture"] = BuiltInCategory.OST_Furniture,
+            ["Generic Models"] = BuiltInCategory.OST_GenericModel,
+            ["Pipes"] = BuiltInCategory.OST_PipeCurves,
+            ["Ducts"] = BuiltInCategory.OST_DuctCurves,
+        };
+
+        public object? Execute(JsonElement args, ToolContext context)
+        {
+            var doc = context.Document ?? throw new NoActiveDocumentException();
+            var info = doc.ProjectInformation;
+            var units = doc.GetUnits();
+
+            string system = doc.DisplayUnitSystem == DisplayUnit.IMPERIAL ? "imperial" : "metric";
+            string? lengthUnit = DisplayUnitName(units, SpecTypeId.Length);
+            var project = new Dictionary<string, object?>
+            {
                 ["title"] = doc.Title,
                 ["documentId"] = DocumentGuard.GetIdentity(doc),
-                ["name"] = info?.Name,
-                ["number"] = info?.Number,
-                ["clientName"] = info?.ClientName,
-                ["address"] = info?.Address,
-                ["buildingName"] = info?.BuildingName,
-                ["author"] = info?.Author,
-                ["filePath"] = doc.PathName,
-                ["revitVersion"] = doc.Application.VersionNumber,
-                ["units"] = new Dictionary<string, object?>
-                {
-                    ["system"] = system,
-                    ["length"] = lengthUnit,
-                    ["area"] = DisplayUnitName(units, SpecTypeId.Area),
-                    ["volume"] = DisplayUnitName(units, SpecTypeId.Volume),
-                },
-            };
-
-            var levels = new FilteredElementCollector(doc)
-                .OfClass(typeof(Level))
-                .Cast<Level>()
-                .OrderBy(level => level.Elevation)
-                .Select(level => new Dictionary<string, object?>
-                {
-                    ["id"] = level.Id.Value,
-                    ["name"] = level.Name,
-                    ["elevation_ft"] = level.Elevation,
-                    ["elevation_display"] = FormatLength(units, level.Elevation),
-                })
-                .ToList();
-
-            var grids = new FilteredElementCollector(doc)
-                .OfClass(typeof(Grid))
-                .Cast<Grid>()
-                .OrderBy(grid => grid.Name, StringComparer.OrdinalIgnoreCase)
-                .Select(grid => new Dictionary<string, object?>
-                {
-                    ["id"] = grid.Id.Value,
-                    ["name"] = grid.Name,
-                })
-                .ToList();
-
-            var counts = new Dictionary<string, int>();
-            foreach (var (label, category) in MajorCategories)
-            {
-                try
-                {
-                    counts[label] = new FilteredElementCollector(doc)
-                        .OfCategory(category)
-                        .WhereElementIsNotElementType()
-                        .GetElementCount();
-                }
-                catch
-                {
-                    counts[label] = 0;
-                }
-            }
-
-            int totalElements = new FilteredElementCollector(doc).WhereElementIsNotElementType().GetElementCount();
-            int sheets = new FilteredElementCollector(doc).OfClass(typeof(ViewSheet)).GetElementCount();
-            int views = new FilteredElementCollector(doc)
-                .OfClass(typeof(View))
-                .Cast<View>()
-                .Count(view => !view.IsTemplate && view is not ViewSheet);
-
-            var totals = new Dictionary<string, int>
-            {
-                ["elements"] = totalElements,
-                ["views"] = views,
-                ["sheets"] = sheets,
-                ["levels"] = levels.Count,
-                ["grids"] = grids.Count,
-            };
-
-            var topCategories = counts
-                .Where(pair => pair.Value > 0)
-                .OrderByDescending(pair => pair.Value)
-                .Take(5)
-                .Select(pair => $"{pair.Key} {pair.Value}")
-                .ToList();
-            string numberPart = string.IsNullOrWhiteSpace(info?.Number) ? string.Empty : $" (project {info!.Number})";
-            string compact = $"'{doc.Title}'{numberPart}: {totalElements} elements, {levels.Count} levels, {grids.Count} grids, {sheets} sheets, {views} views; units {system} ({lengthUnit})."
-                + (topCategories.Count > 0 ? $" Top categories: {string.Join(", ", topCategories)}." : string.Empty);
-
-            return new ToolOutput(new { project, levels, grids, counts, totals }, compact);
-        }
-
-        private static string? DisplayUnitName(Units units, ForgeTypeId spec)
-        {
-            try
-            {
-                return UnitResolver.ShortName(units.GetFormatOptions(spec).GetUnitTypeId());
-            }
-            catch
-            {
-                return null;
-            }
-        }
-
-        private static string? FormatLength(Units units, double value)
-        {
-            try
-            {
-                return UnitFormatUtils.Format(units, SpecTypeId.Length, value, false);
-            }
-            catch
-            {
-                return null;
-            }
-        }
-    }
-}
+                ["name"] = info?.Name,
+                ["number"] = info?.Number,
+                ["clientName"] = info?.ClientName,
+                ["address"] = info?.Address,
+                ["buildingName"] = info?.BuildingName,
+                ["author"] = info?.Author,
+                ["filePath"] = doc.PathName,
+                // Project or family: tools declare which kinds they work in, and the bridge refuses others.
+                ["documentKind"] = DocumentGuard.KindOf(doc),
+                ["revitVersion"] = doc.Application.VersionNumber,
+                ["units"] = new Dictionary<string, object?>
+                {
+                    ["system"] = system,
+                    ["length"] = lengthUnit,
+                    ["area"] = DisplayUnitName(units, SpecTypeId.Area),
+                    ["volume"] = DisplayUnitName(units, SpecTypeId.Volume),
+                },
+            };
+
+            var levels = new FilteredElementCollector(doc)
+                .OfClass(typeof(Level))
+                .Cast<Level>()
+                .OrderBy(level => level.Elevation)
+                .Select(level => new Dictionary<string, object?>
+                {
+                    ["id"] = level.Id.Value,
+                    ["name"] = level.Name,
+                    ["elevation_ft"] = level.Elevation,
+                    ["elevation_display"] = FormatLength(units, level.Elevation),
+                })
+                .ToList();
+
+            var grids = new FilteredElementCollector(doc)
+                .OfClass(typeof(Grid))
+                .Cast<Grid>()
+                .OrderBy(grid => grid.Name, StringComparer.OrdinalIgnoreCase)
+                .Select(grid => new Dictionary<string, object?>
+                {
+                    ["id"] = grid.Id.Value,
+                    ["name"] = grid.Name,
+                })
+                .ToList();
+
+            var counts = new Dictionary<string, int>();
+            foreach (var (label, category) in MajorCategories)
+            {
+                try
+                {
+                    counts[label] = new FilteredElementCollector(doc)
+                        .OfCategory(category)
+                        .WhereElementIsNotElementType()
+                        .GetElementCount();
+                }
+                catch
+                {
+                    counts[label] = 0;
+                }
+            }
+
+            int totalElements = new FilteredElementCollector(doc).WhereElementIsNotElementType().GetElementCount();
+            int sheets = new FilteredElementCollector(doc).OfClass(typeof(ViewSheet)).GetElementCount();
+            int views = new FilteredElementCollector(doc)
+                .OfClass(typeof(View))
+                .Cast<View>()
+                .Count(view => !view.IsTemplate && view is not ViewSheet);
+
+            var totals = new Dictionary<string, int>
+            {
+                ["elements"] = totalElements,
+                ["views"] = views,
+                ["sheets"] = sheets,
+                ["levels"] = levels.Count,
+                ["grids"] = grids.Count,
+            };
+
+            var topCategories = counts
+                .Where(pair => pair.Value > 0)
+                .OrderByDescending(pair => pair.Value)
+                .Take(5)
+                .Select(pair => $"{pair.Key} {pair.Value}")
+                .ToList();
+            string numberPart = string.IsNullOrWhiteSpace(info?.Number) ? string.Empty : $" (project {info!.Number})";
+            string compact = $"'{doc.Title}'{numberPart}: {totalElements} elements, {levels.Count} levels, {grids.Count} grids, {sheets} sheets, {views} views; units {system} ({lengthUnit})."
+                + (topCategories.Count > 0 ? $" Top categories: {string.Join(", ", topCategories)}." : string.Empty);
+
+            if (doc.IsFamilyDocument) project["family"] = DescribeFamily(doc);
+            return new ToolOutput(new { project, levels, grids, counts, totals }, compact);
+        }
+
+        /// <summary>What a family document holds: its category, types and parameters (names only).</summary>
+        private static Dictionary<string, object?> DescribeFamily(Document doc)
+        {
+            var manager = doc.FamilyManager;
+            var category = doc.OwnerFamily?.FamilyCategory;
+            return new Dictionary<string, object?>
+            {
+                ["category"] = category?.Name,
+                ["builtInCategory"] = category?.BuiltInCategory.ToString(),
+                ["currentType"] = manager.CurrentType?.Name,
+                ["types"] = manager.Types.Cast<FamilyType>().Select(type => type.Name).OrderBy(name => name).ToList(),
+                ["parameters"] = manager.Parameters.Cast<FamilyParameter>()
+                    .Select(parameter => new { name = parameter.Definition.Name, instance = parameter.IsInstance, formula = parameter.Formula })
+                    .OrderBy(parameter => parameter.name).ToList(),
+                ["api"] = "Family types, parameters and formulas are edited through FamilyManager (" + DocumentKind.FamilyApi + ") with execute_csharp; project-only tools refuse this document.",
+            };
+        }
+
+        private static string? DisplayUnitName(Units units, ForgeTypeId spec)
+        {
+            try
+            {
+                return UnitResolver.ShortName(units.GetFormatOptions(spec).GetUnitTypeId());
+            }
+            catch
+            {
+                return null;
+            }
+        }
+
+        private static string? FormatLength(Units units, double value)
+        {
+            try
+            {
+                return UnitFormatUtils.Format(units, SpecTypeId.Length, value, false);
+            }
+            catch
+            {
+                return null;
+            }
+        }
+    }
+}

package/src/Revit/Tools/GetScheduleFields.cs

@@ -1,37 +1,44 @@
-using System.Text.Json;
-using Autodesk.Revit.DB;
-
-namespace RevitBridge.Tools;
-
-internal sealed class GetScheduleFields : ITool
-{
-    public string Name => "get_schedule_fields";
-    public string Label => "Get Schedule Fields";
-    public string Tier => "advanced";
-    public string Description => "List fields eligible to be added to one existing schedule, with paging and localized-name filtering. A field is identified by the pair parameter_id + field_type; negative built-in parameter IDs are valid. These identities differ from schedule-local field_id values returned by get_schedules. Existing fields are marked included. Count may appear in the returned fields; manage_schedules accepts its discovered pair or field_type Count without a parameter_id. Calculated/combined fields are not exposed here.";
-    public object ParametersSchema => new
-    {
-        type = "object", properties = new
-        {
-            schedule_id = new { type = "integer", minimum = 1 }, name_filter = new { type = "string" },
-            offset = new { type = "integer", minimum = 0 }, limit = new { type = "integer", minimum = 1, maximum = 200 },
-        }, required = new[] { "schedule_id" },
-    };
-    public object Execute(JsonElement args, ToolContext context)
-    {
-        var doc = context.Document ?? throw new NoActiveDocumentException();
-        var schedule = doc.GetElement(new ElementId(JsonArgs.GetLong(args, "schedule_id") ?? 0)) as ViewSchedule ?? throw new ArgumentException("schedule_id is not a schedule.");
-        if (schedule.IsTemplate || schedule.IsTitleblockRevisionSchedule) throw new ArgumentException("Use a regular schedule, not a template or titleblock revision schedule.");
-        int offset = JsonArgs.GetInt(args, "offset", 0), limit = JsonArgs.GetInt(args, "limit", 100);
-        if (offset < 0 || limit is < 1 or > 200) throw new ArgumentException("Invalid paging range.");
-        string needle = JsonArgs.GetString(args, "name_filter") ?? "";
-        var definition = schedule.Definition;
-        var included = definition.GetFieldOrder().Select(id => definition.GetField(id)).Select(f => (f.ParameterId.Value, f.FieldType)).ToHashSet();
-        var fields = definition.GetSchedulableFields().Select(f => new
-        {
-            parameter_id = f.ParameterId.Value, field_type = f.FieldType.ToString(), name = f.GetName(doc), included = included.Contains((f.ParameterId.Value, f.FieldType)),
-        }).Where(f => f.name.Contains(needle, StringComparison.OrdinalIgnoreCase)).OrderBy(f => f.parameter_id).ThenBy(f => f.field_type, StringComparer.Ordinal).ToArray();
-        var page = fields.Skip(offset).Take(limit).ToArray();
-        return new { schedule_id = schedule.Id.Value, total_count = fields.Length, offset, returned_count = page.Length, next_offset = offset + page.Length < fields.Length ? (int?)(offset + page.Length) : null, fields = page };
-    }
-}
+using System.Text.Json;
+using Autodesk.Revit.DB;
+
+namespace RevitBridge.Tools;
+
+internal sealed class GetScheduleFields : ITool
+{
+    public string Name => "get_schedule_fields";
+    public IReadOnlyList<string> DocumentKinds => DocumentKind.ProjectOnly;
+    public IReadOnlyList<string> Keywords => new[] { "schedule fields", "available fields", "add field", "schedulable parameters", "column" };
+    public IReadOnlyList<ToolLimit> Limits => new[]
+    {
+        new ToolLimit("Calculated (formula) fields", "revit_unsupported", "Revit 2025 API exposes ScheduleFieldType.Formula but no member to author a formula"),
+        new ToolLimit("Combined-parameter fields", "api", "ScheduleDefinition.InsertCombinedParameterField"),
+    };
+    public string Label => "Get Schedule Fields";
+    public string Tier => "advanced";
+    public string Description => "List fields eligible to be added to one existing schedule, with paging and localized-name filtering. A field is identified by the pair parameter_id + field_type; negative built-in parameter IDs are valid. These identities differ from schedule-local field_id values returned by get_schedules. Existing fields are marked included. Count may appear in the returned fields; manage_schedules accepts its discovered pair or field_type Count without a parameter_id. Calculated/combined fields are not exposed here.";
+    public object ParametersSchema => new
+    {
+        type = "object", properties = new
+        {
+            schedule_id = new { type = "integer", minimum = 1 }, name_filter = new { type = "string" },
+            offset = new { type = "integer", minimum = 0 }, limit = new { type = "integer", minimum = 1, maximum = 200 },
+        }, required = new[] { "schedule_id" },
+    };
+    public object Execute(JsonElement args, ToolContext context)
+    {
+        var doc = context.Document ?? throw new NoActiveDocumentException();
+        var schedule = doc.GetElement(new ElementId(JsonArgs.GetLong(args, "schedule_id") ?? 0)) as ViewSchedule ?? throw new ArgumentException("schedule_id is not a schedule.");
+        if (schedule.IsTemplate || schedule.IsTitleblockRevisionSchedule) throw new ArgumentException("Use a regular schedule, not a template or titleblock revision schedule.");
+        int offset = JsonArgs.GetInt(args, "offset", 0), limit = JsonArgs.GetInt(args, "limit", 100);
+        if (offset < 0 || limit is < 1 or > 200) throw new ArgumentException("Invalid paging range.");
+        string needle = JsonArgs.GetString(args, "name_filter") ?? "";
+        var definition = schedule.Definition;
+        var included = definition.GetFieldOrder().Select(id => definition.GetField(id)).Select(f => (f.ParameterId.Value, f.FieldType)).ToHashSet();
+        var fields = definition.GetSchedulableFields().Select(f => new
+        {
+            parameter_id = f.ParameterId.Value, field_type = f.FieldType.ToString(), name = f.GetName(doc), included = included.Contains((f.ParameterId.Value, f.FieldType)),
+        }).Where(f => f.name.Contains(needle, StringComparison.OrdinalIgnoreCase)).OrderBy(f => f.parameter_id).ThenBy(f => f.field_type, StringComparer.Ordinal).ToArray();
+        var page = fields.Skip(offset).Take(limit).ToArray();
+        return new { schedule_id = schedule.Id.Value, total_count = fields.Length, offset, returned_count = page.Length, next_offset = offset + page.Length < fields.Length ? (int?)(offset + page.Length) : null, fields = page };
+    }
+}