Repository navigation
Establish the AI-first base for developerz-ai/mcp-ssh - #41
developerz-ai[bot] wants to merge 4 commits into
Conversation
Dz-Task-Id: tsk_a9e8580adb2da18c3ff9190a7a3a69a5
✅ Reviewed — nothing blocking0 actionable comment(s) · grounded on your code ⏱ 1m 24s wall clock · 🤖 developerz.ai — automated review, running on your model and your box. |
There was a problem hiding this comment.
Review summary — 18 file(s), 9 finding(s).
Critical 0 · Major 0 · Minor 7 · Nit 2
Onboarding PR adds machine-read policy files, scorecard and schedules. The new reviewer.md is a reviewer-instruction file full of TODO placeholders and quotes a gate command that contradicts the environment's mise-based toolchain. Onboarding-only PR adding agent definitions and skills; no code.
| File | Findings |
|---|---|
.claude/agents/oauth-pkce.md |
1 nit |
.claude/skills/bearer-token-mint/SKILL.md |
1 minor |
.claude/skills/verify-change/SKILL.md |
3 minor |
.dz/maintainer/maintainer.yml |
1 minor |
.dz/maintainer/reviewer.md |
2 minor |
docs/README.md |
1 nit |
- the pull request title was withheld by the untrusted-input screen (injection) — the reviewer did not read it. This pull request was opened by this platform's own App, so this alone does not change the review's verdict
- test files were not shown to the reviewer — this diff touched no path recognised as a test, so no assertion was available to read the change against
🤖 developerz.ai review — automated, running on your model and your box. What is this? · Reviewed by router/glm-5.3-flash on box hv-dedicated-dev-marshall — 351.5k in / 7.81k out over 10 calls, on your flat-rate plan: no per-token charge (~$0.06 at list price).
|
Address review findings on #41
Work landed on #41 — watching it toward merge. 10m · 58 tool calls · 3 messages Sessions: coding run · review 1 · review 2 · review 3 🤖 developerz.ai, an automated agent, not a human — hv-dedicated-2 (developerz-ai[bot]) · model |
|
Waiting on the repository owner: this pull request changes a gate file, which the platform does not merge by itself. It changes The merge check refused the edit: .dz/maintainer/reviewer.md: a new file that differs from the platform template rendered for it for #41 — proof path: The task it answers planned this edit, so the goal cannot be met without it and it is not re-filed without them. To take it, merge it on GitHub. To have the platform merge it, push a narrower edit: a new head is checked again and merged when the check passes. Nothing here is re-tried or closed automatically, and no email is sent; the pull request is listed with the stuck pull requests. 🤖 Posted by developerz.ai — an automated agent, not a human. |
Follow-up intent: Address review findings on #41 Dz-Task-Id: tsk_1913232f9809f78aef3167ac2bdf15a1
There was a problem hiding this comment.
Review summary — 6 file(s), 0 finding(s).
Docs-only increment: fills the reviewer.md placeholders, corrects the release mechanism in maintainer.yml, rewrites the verify-change exit-code contract to match the actual bin/check chain, and converts the token-mint step to env-injection — all resolving prior review findings; no new defects visib…
- the pull request title was withheld by the untrusted-input screen (injection) — the reviewer did not read it. This pull request was opened by this platform's own App, so this alone does not change the review's verdict
- test files were not shown to the reviewer — this diff touched no path recognised as a test, so no assertion was available to read the change against
🤖 developerz.ai review — automated, running on your model and your box. What is this? · Reviewed by router/glm-5.3-flash on box hv-dedicated-dev-marshall — 54.8k in / 728 out over 2 calls, on your flat-rate plan: no per-token charge (~<$0.01 at list price).
…ge proof can pass The babysit follow-up 0cca4e7 rewrote .dz/maintainer/reviewer.md and .dz/maintainer/maintainer.yml. The merge egress has refused this PR as gate_path_edit_unproven since, because its proof needs these paths to match the template's bytes. This restores both files to the setup commit 1693cc2 and leaves every non-gate edit from 0cca4e7 as it is. The lane fix is developerz-ai/developerz.ai#8405. The model-written reviewer rules belong in a separate PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Restored |
There was a problem hiding this comment.
Review summary — 2 file(s), 3 finding(s).
Critical 0 · Major 1 · Minor 2 · Nit 0
This increment reverts the accurate, repo-specific maintainer/reviewer content added earlier in this PR back to generic onboarding boilerplate: the release mechanism is again described as unobserved and the reviewer file's conventions, gates and never-flag rules are back to TODO(setup).
- the pull request title was withheld by the untrusted-input screen (injection) — the reviewer did not read it. This pull request was opened by this platform's own App, so this alone does not change the review's verdict
- test files were not shown to the reviewer — this diff touched no path recognised as a test, so no assertion was available to read the change against
🤖 developerz.ai review — automated, running on your model and your box. What is this? · Reviewed by router/deepseek-v4.1-flash on box sebastian-1 — 22.1k in / 3.65k out over 2 calls, on your flat-rate plan: no per-token charge (~<$0.01 at list price).
|
No code change warranted. The No change warranted — reviewer feedback on #41 is already handled at headFeedback (@sebyx07): What I checked at head
Why no editThe requested end state is the current end state: both gate files already hold the setup The two follow-on items in the feedback are outside this working tree by design, not
No defect class in this repository produced the rewrite, so there is nothing here to guard. Posted automatically by the developerz bot; the reasoning above is the follow-up run's own (AI-authored). |
Follow-up intent: Address review findings on #41 Dz-Task-Id: tsk_3d76830814acd24e2ee6d8120d14cc4a
There was a problem hiding this comment.
Review summary — 1 file(s), 0 finding(s).
This increment replaces the three TODO(setup) placeholders in .dz/maintainer/reviewer.md with substantive review conventions, gate definitions and never-flag rules mirrored from CLAUDE.md/reviewer instructions. Docs-only, no code touched, no defect I can anchor against the repo at this head.
- the pull request title was withheld by the untrusted-input screen (injection) — the reviewer did not read it. This pull request was opened by this platform's own App, so this alone does not change the review's verdict
- test files were not shown to the reviewer — this diff touched no path recognised as a test, so no assertion was available to read the change against
🤖 developerz.ai review — automated, running on your model and your box. What is this? · Reviewed by router/deepseek-v4.1-flash on box hv-dedicated-dev-marshall — 43k in / 14.8k out over 2 calls, on your flat-rate plan: no per-token charge (~$0.02 at list price).
This pull request establishes the base an AI coding agent needs to work well in
developerz-ai/mcp-ssh.Stack
The account's model ran this repository and answered its stack: Rust + Cargo.
Established
bin/stop— 4. scripts that work.claude/agents/job-supervision.md— 2. a specialist roster.claude/agents/oauth-pkce.md— 2. a specialist roster.claude/agents/sqlite-persistence.md— 2. a specialist roster.claude/agents/mcp-tools.md— 2. a specialist roster.claude/skills/verify-change/SKILL.md— 3. project skills.claude/skills/sqlite-migrations/SKILL.md— 3. project skills.claude/skills/release-cut/SKILL.md— 3. project skills.claude/skills/bearer-token-mint/SKILL.md— 3. project skillsAGENTS.mddocs/README.md.githooks/pre-commit— 4. scripts that work.mcp.json— 5. agent wiring.dz/maintainer/maintainer.yml— 0. the repo policy.dz/schedules/hygiene.md.dz/schedules/devops.md.dz/maintainer/reviewer.md— 0. the repo policy.dz/onboarding/scorecard.mdLeft alone
These already existed and are yours — nothing here was rewritten, reordered or reflowed:
bin/setup— 4. scripts that workbin/dev— 4. scripts that workbin/check— 4. scripts that workCLAUDE.md— 1. a brain.env.example.claude/commands— 9. workflow commands.github/workflows/ci.yml— 4. scripts that work.gitignoreConfigured nowhere / not needed by the gate
Read by the code, set nowhere this run touched, and never named by a failed gate run — so they are dormant integrations or ops-only entries that never blocked anything. The list is bounded to ten to keep the body short:
MCP_SSH_PASS— read by bin/mcp-token (+2 more)SENTRY_DSN— read by src/sentry.rsMCP_SSH_BIND— read by bin/mcp-token (+3 more)MCP_SSH_ALLOWED_HOSTS— read by deploy/install.sh (+1 more)MCP_SSH_USER— read by bin/mcp-token (+1 more)MCP_SSH_INLINE_TIMEOUT_SECS— read by src/config.rsMCP_SSH_JOB_DIR— read by src/config.rsRUST_LOG— read by .env.exampleSENTRY_ENVIRONMENT— read by src/sentry.rsVerification
The gate this pull request declares, run on the box that wrote it:
bin/setupthenbin/check, before these changes (base) and after them (head); thenbin/stop, the teardown the platform runs when every session ends; thenbin/setupandbin/stoponce more on the same box (reuse), the way every later session meets it.Proof: unprovable —
bin/checkdid not run green at head on this box. Nothing in the repository’s own code was changed to force it; the excerpt below is the record.Scorecard
10 of 15 items ok. The whole card, with every piece of evidence, is
.dz/onboarding/scorecard.md.policy— ok — .dz/maintainer/maintainer.yml establishedbrain— ok — CLAUDE.md already present, left aloneroster— ok — .claude/agents/job-supervision.md establishedskills— ok — .claude/skills/verify-change/SKILL.md establishedscripts— unprovable — bin/setup already present, left alonewiring— ok — .mcp.json establishedcommands— ok — .claude/commands already present, left aloneaccess— gap — landed 0 of 9 askedlayout— gap — migration to the house layout filed as taskssmoke_test— unprovable — src/app_tests.rs in the repositoryenv_example— ok — .env.example already present, left aloneagents_md— ok — AGENTS.md establishedper_workspace_brains— ok — single-package repository: 0 workspacesdocs_readme— ok — docs/README.md establishedgate_proved— unprovable — ./bin/check at head: red, exit 1 (0.0s)First tasks
Queued from what this pass found, and none of them runs before this merges:
Merging
This rewrites how agents read your repo, on a repo the platform has not worked before,
so it is yours to merge — not the bot's. Merging it is also what marks the repo
onboarded; until then coding tasks on it are held (issue triage, PR review and
report-only tasks are unaffected).
🤖 Opened by developerz.ai for task
tsk_a9e8580adb2da18c3ff9190a7a3a69a5on developerz-ai/mcp-ssh.Raised automatically by the
repo-onboardinglane, from an interactive repository setup session.