Skip to content

feat: add JSON Schema documentation, ESLint config, and MCP tooling improvements#23

Merged
rothnic merged 6 commits intomainfrom
fix/limit-tools-for-opencode
Nov 19, 2025
Merged

feat: add JSON Schema documentation, ESLint config, and MCP tooling improvements#23
rothnic merged 6 commits intomainfrom
fix/limit-tools-for-opencode

Conversation

@rothnic
Copy link
Copy Markdown
Owner

@rothnic rothnic commented Nov 19, 2025

Summary

  • Add comprehensive JSON Schema guide for API/MCP workflows
  • Configure ESLint for ES modules with proper ignore patterns
  • Add MCP normalization script and tool categorization
  • Update ls-lint configuration to exclude .opencode directory
  • Fix unused variable linting errors across codebase

Changes

Documentation

  • docs/json-schema-manual-guide.md - Complete guide for manual JSON Schema authoring
  • docs/windmill-scripts-guide.md - Windmill scripts guide with schema warnings
  • docs/generated-tools.md - Auto-generated tools list
  • docs/agent-setup-complete.md - Agent setup documentation
  • docs/testing-setup-guide.md - Testing setup guide
  • docs/windmill-agent-team-plan.md - Agent team planning documentation

Configuration

  • .eslintrc.json - ESLint configuration for ES2022 modules
  • .eslintignore - Ignore build/ and node_modules/
  • .ls-lint.yml - Updated to ignore .opencode directory and allow ESLint config

Code Quality

  • Fixed unused variable errors in generator, runtime, and test files
  • Removed unused imports and parameters

New Features

  • scripts/normalize-and-run-mcp.js - MCP server normalization script
  • src/utils/tool-categories.js - Tool categorization utility
  • src/overrides/tsconfig.patch.json - TypeScript configuration override

Testing

  • ✅ All unit tests passing
  • ✅ ESLint passing with no errors
  • ✅ ls-lint passing with updated configuration

@rothnic
Copy link
Copy Markdown
Owner Author

rothnic commented Nov 19, 2025

@copilot please fix the build issues

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 19, 2025

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

Copilot AI and others added 2 commits November 18, 2025 21:22
* Initial plan

* fix: update CI workflow paths from generator/ to src/generator/

Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>
@rothnic
Copy link
Copy Markdown
Owner Author

rothnic commented Nov 19, 2025

@copilot still failing, please fix the build issues

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 19, 2025

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

* Initial plan

* fix: update CI workflow to check correct build output path (build/build/index.js)

Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>

* fix: add empty JSON object to fix-schemas.json to fix validation

Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>

* fix: restore tsconfig and package.json overrides to ensure build/dist/index.js output

Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rothnic <452052+rothnic@users.noreply.github.com>
@rothnic rothnic merged commit 799c6a5 into main Nov 19, 2025
3 checks passed
@rothnic rothnic deleted the fix/limit-tools-for-opencode branch November 19, 2025 18:27
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.

3 participants