MCP, MCP server
io.github.Red-Square-Software/keeping-mcp
Open-source Model Context Protocol (MCP) server that exposes the Keeping (api.keeping.nl) time-tracking API as tools an AI coding assistant can call.
Compatibility: Not stated.
Grade B
Minor issues only, no Medium or High. Worth a quick look at the findings below before installing.
Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.
Rescanned at commit 41e964ae601b, unchanged since the check below.
NETWORK_NO_FIXED_HOST SECRETS_PATH_DOTENV OBFUSCATION_ENCODED_BLOB PERMISSION_SHELL_DECLARED SECRETS_ENV_READ
Why this grade
- NETWORK_NO_FIXED_HOST Low
Makes a network call whose host is not written as a fixed address.
scripts/probe-live.ts, line 118
const res = await fetch(url, { - SECRETS_PATH_DOTENV Low
Mentions a .env file.
scripts/check-publish-shape.ts, line 9
// .env, a forgotten dist subdirectory) fails before publish. - OBFUSCATION_ENCODED_BLOB Low
Contains a long base64 shaped string in a file that should be readable.
.planning/research/keeping-openapi.json, line 1
{"openapi":"3.0.0","info":{"title":"Keeping API Documentation","version":"v1","x-logo":{"url":"https:\/\/developer.keeping.nl\/images\/logo.svg","altText":"Keeping logo"},"descrip - PERMISSION_SHELL_DECLARED Note
Runs shell commands, and the manifest or description says so.
.planning/STATE.md, line 127
| Windows-first README ordering (Plan 04-02, RESEARCH §Pitfall 2) | The README's `## Install` section places the Windows config block (`command: "cmd", args: ["/c", "npx", "-y", "k - SECRETS_ENV_READ Note
Reads an environment variable whose name contains KEY, SECRET, TOKEN, PASSWORD or CREDENTIAL.
scripts/probe-live.ts, line 414
if (!process.env.KEEPING_TOKEN) {
Any High, or two or more Medium findings, give D. One Medium gives C. Low findings or a security Note give B. Each code names a finding behind the letter.
Grades are a hygiene check of the source at the commit shown, on the date checked, not a security audit or a guarantee. The free directory grades are automated. Before release, a PlugVet engineer reviews every paid report and any grade we hold back, including all F grades, against the source code and the evidence we quote.
What was checked
Plain notes from the hygiene checks.
Permissions
Note
requests shell access. The manifest names this permission.
PERMISSION_SHELL_DECLARED
.planning/STATE.md, line 127
| Windows-first README ordering (Plan 04-02, RESEARCH §Pitfall 2) | The README's `## Install` section places the Windows config block (`command: "cmd", args: ["/c", "npx", "-y", "k
.planning/phases/02-read-tools-schema-discovery/02-RESEARCH.md, line 613
| Spawning the built binary for unit tests | `child_process.spawn` + parse JSON-RPC frames from stdout | `InMemoryTransport.createLinkedPair()` | First-party SDK pattern, no proces
.planning/phases/04-distribution-release-pipeline/04-RESEARCH.md, line 363
**What:** Windows users adding the server to Claude Code MUST wrap `npx` with `cmd /c`. Bare `npx` fails with `spawn npx ENOENT` because Node's `child_process.spawn` does not resol
Show more
.planning/phases/04-distribution-release-pipeline/04-RESEARCH.md, line 428
**Why it happens:** Documented at length in anthropics/claude-code#58510 - Node's `child_process.spawn` on Windows doesn't resolve `.cmd` extensions via PATHEXT unless `shell: true
.planning/research/PITFALLS.md, line 151
On Windows, `npx` resolves to `npx.cmd`, a batch script. Node's `child_process.spawn()` does not invoke `.cmd` files as executables - it expects a binary. Claude Code uses spawn in
Install and update
Pass
No install-time script found, and dependencies are pinned or absent.
Network
Concern
can reach a host that is not written as a fixed address.
NETWORK_NO_FIXED_HOST
scripts/probe-live.ts, line 118
const res = await fetch(url, {
scripts/probe-live.ts, line 153
const res = await fetch(`${KEEPING_BASE}${path}`, {
scripts/probe-live.ts, line 445
const orgsRawRes = await fetch(`${KEEPING_BASE}/organisations`, {
Show more
scripts/probe-live.ts, line 488
// Q1 contingency evidence - raw fetch (NOT client.me()) so a 401 doesn't
scripts/probe-live.ts, line 494
const meRes = await fetch(`${KEEPING_BASE}/users/me`, {
scripts/probe-live.ts, line 507
const meOrgRes = await fetch(`${KEEPING_BASE}/${orgId}/users/me`, {
src/keeping/client.ts, line 241
const res = await fetch(`${BASE}${path}`, {
src/keeping/client.ts, line 283
const res = await fetch(`${BASE}${path}`, {
.planning/phases/02-read-tools-schema-discovery/02-05-PLAN.md, line 318
const meRes = await fetch("https://api.keeping.nl/v1/users/me", {
Secrets
Concern
reads a credential or environment file.
SECRETS_ENV_READ SECRETS_PATH_DOTENV
scripts/check-publish-shape.ts, line 9
// .env, a forgotten dist subdirectory) fails before publish.
scripts/probe-live.ts, line 414
if (!process.env.KEEPING_TOKEN) {
scripts/probe-live.ts, line 416
"[probe-live] KEEPING_TOKEN must be set in env or .env before running probe-live\n",
Show more
.gitignore, line 3
coverage/
.gitignore, line 4
.env
.planning/MILESTONES.md, line 16
- The one-shot live-API probe ships in tree: `scripts/probe-live.ts` (440 lines) bundles a fully-tested `anonymise()` denylist walker (9/9 tests, including the `ANONYMISE_KEYS.size
.planning/STATE.md, line 112
| Probe-live pre-check + loadConfig double layer (Plan 02-05) | Script emits byte-identical `[probe-live] KEEPING_TOKEN must be set in env or .env before running probe-live` to std
.planning/milestones/v1.0-ROADMAP.md, line 135
1. `npm pack --dry-run` output contains only `dist/`, `README.md`, and `LICENSE` - no `.env`, test fixtures, `.github/`, or dotfiles appear; the `files` whitelist in `package.json`
.planning/phases/01-foundation-scaffolding/01-01-PLAN.md, line 67
provides: "Ignores node_modules/, dist/, coverage/, .env*, *.log, *.tsbuildinfo, .idea/ (D-21)"
Agent instructions
Pass
No instruction to ignore earlier instructions or to hide a step.
Obfuscation
Concern
contains an encoded blob in a file that should be readable.
OBFUSCATION_ENCODED_BLOB
.planning/research/keeping-openapi.json, line 1
{"openapi":"3.0.0","info":{"title":"Keeping API Documentation","version":"v1","x-logo":{"url":"https:\/\/developer.keeping.nl\/images\/logo.svg","altText":"Keeping logo"},"descrip
Licence
Pass
A licence file is present (LICENSE).
Maintainer activity
Pass
The latest commit is within the last 365 days.
Note
Hygiene check of commit 41e964ae601b. requests shell access. The manifest names this permission. can reach a host that is not written as a fixed address. reads a credential or environment file. contains an encoded blob in a file that should be readable.
Date and source
Date checked:
github.com
Commit graded: 41e964ae601bebd2c1ec9f5211a4e0922ba80af8
This link goes to the original host. PlugVet does not host or check what you download from there, and installing it is your own decision and risk.