Skip to content

Misc improvements#52

Merged
a-bonus merged 14 commits intoa-bonus:mainfrom
davidthor:vitest-transformer-lean-tools
Feb 12, 2026
Merged

Misc improvements#52
a-bonus merged 14 commits intoa-bonus:mainfrom
davidthor:vitest-transformer-lean-tools

Conversation

@davidthor
Copy link
Copy Markdown
Contributor

Since you were amenable to my last round of changes, I thought I'd see if you were interested in the latest batch. Here's what's included here:

I know this is a HUGE change, and I'm sorry about that. I started working on this when I wasn't sure if you'd be open to PRs. I'll make smaller contributions in the future.

New features

  • Added support for freezing rows/columns in google sheets
  • Added ability to set/unset dropdown validation rules on cells in google sheets

Other improvements and bug fixes

  • Improved grid range specification for editing cells in google sheets
  • Stripped tool calls that weren't implemented.
    • Too many tools decreases AIs ability to select the correct tools
  • Renamed tools and updated descriptions to improve legibility in agent execution, and improve agent effectiveness
  • Fixed bugs with markdown insertion into docs
    • I wasn't able to get my agent to convert the markdown for me on insert
    • Created a dedicated module for converting to/from markdown (src/markdown-transformer)
  • Setup npm packaging workflows
    • Triggered on semvar tag/release creation
    • Builds and publishes to NPM to avoid the need for people to clone the repo
    • Accepts oauth app client id and secret via environment variables
    • Stores token.json in the user root instead of the cloned repository location so it works with npm package installs
    • Split contributing guides, including git clone flow, into contributing docs instead of main readme
    • Note: it'll be up to you to decide if/when to cut proper "releases" to trigger this workflow. The current dev flow still works except for the new location of the token.json file, but the authorization flow puts it in the new location automatically.
  • Use vitest instead of node-native for tests
    • Primarily so that tests get run on the typescript instead of the built node code. Vitest is marginally faster as well, but not the point of the change given the small footprint of the project.
  • Move tests to live alongside the files they're testing now that the root src directory is more complex with nested modules.
  • Added logger utility that responds to different log level configurations
    • Logs from utilities were bloating the results of the test execution for expected behavior
    • A huge amount of the actual file changes are patches to use this utility
  • Added debugging details to insert doc content tool call, making it easier for the agent to let us know why requests take longer than usual or where things failed.

@a-bonus a-bonus merged commit 65f8e33 into a-bonus:main Feb 12, 2026
2 checks passed
@a-bonus
Copy link
Copy Markdown
Owner

a-bonus commented Feb 12, 2026

you're the goat for this level of contribution!

@davidthor
Copy link
Copy Markdown
Contributor Author

Thanks for putting in the work to get the project off the ground! Made it easier for me to contribute to!

@davidthor davidthor deleted the vitest-transformer-lean-tools branch February 12, 2026 19:24
@stefanotorresi
Copy link
Copy Markdown

The google-docs-mcp npm package doesn't seem to be owned by this project atm and the indication to use npx google-docs-mcp will lead people to install and run the wrong thing!

This has huge security implications, as you might imagine.

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