Skip to content

5HNb3hh4eAbqqhdsoeJuw1UFaLix22DtyNfow2mvaBEDHEhs update - #1582

Open
duelwarrior630-crypto wants to merge 1 commit into
unarbos:mainfrom
duelwarrior630-crypto:miner/5HNb3hh4-1778780644
Open

duelwarrior630-crypto wants to merge 1 commit into
unarbos:mainfrom
duelwarrior630-crypto:miner/5HNb3hh4-1778780644

Conversation

@duelwarrior630-crypto

Copy link
Copy Markdown

Summary

Miner submission for Subnet 66.
Hotkey: 5HNb3hh4eAbqqhdsoeJuw1UFaLix22DtyNfow2mvaBEDHEhs
Branch: miner/5HNb3hh4-1778780644 from duelwarrior630-crypto/ninja-miner
Author note: update

Scope

  • Diff size: +51 / -5 (56 changed lines)
  • Functions modified (3): _check_json_syntax_one, _should_skip_patch_path, build_self_check_prompt

@github-actions

Copy link
Copy Markdown

OpenRouter PR Judge

Verdict: FAIL
Model: anthropic/claude-opus-4.7
Threshold: 70

Score Value
Overall 58
Real edit 55
Safety 95
Scope 80
Contract 100

Summary

The PR makes three small changes: (1) expands the list of file extensions/names/directories skipped in patch generation (.swp, .DS_Store, .tox, .idea, .vscode, .next, .turbo, etc.), (2) adds .dart to the brace-balance syntax check suffix set, and (3) appends two new prose sections to the system prompt ("PRESERVE SIBLING IDENTIFIERS", "TASK SCOPE LOCK") and two new sections inside build_self_check_prompt ("UNDEFINED-IDENTIFIER HARD STOP", "DELIVERABLE COVERAGE").

Static Checks

  • No static findings.

Judge Reasons

  • Mechanical changes are minor but real: the path-skip list and brace-balance language list are genuine, observable behavior tweaks that affect patch sanitization and syntax checking.
  • The bulk of the diff (~40 of 51 added lines) is prose-only additions to SYSTEM_PROMPT and build_self_check_prompt with no accompanying mechanism change.
  • No contract drift, no third-party imports, no sampling-param additions, no provider hostname leakage; DANGEROUS_PATTERNS untouched.
  • No obfuscation, no removed markers, no encoded payloads. Identifiers and comments remain readable.
  • The added prompt text includes some judge-/scope-adjacent framing ("costs you on similarity", deliverable coverage checklist) which leans toward Goodhart-style prompt shaping rather than solver mechanics.

Risks

  • goodhart: new prompt section explicitly references 'similarity AND scope-creep penalty', nudging the inner model toward matching the reference patch shape rather than purely solving the issue.
  • prompt-only-tuning: majority of the diff is prose appended to SYSTEM_PROMPT / self-check prompt without a corresponding mechanism change; the two mechanical tweaks (path skip, brace suffix) are very small.

Required Changes

  • Pair the prompt additions with a real mechanical change (e.g., actually implement an undefined-identifier check or deliverable-coverage check in code), or trim the prompt additions to keep the PR focused on the path-skip / brace-balance mechanical improvements.
  • Remove or soften the prompt language that explicitly invokes the scoring layer ('costs you on similarity'); frame guidance in terms of correctness, not scoring.

@unarbos unarbos added the notice: missing-commitment Validator notice that no matching on-chain PR commitment was found. label May 14, 2026
@unarbos

unarbos commented May 14, 2026

Copy link
Copy Markdown
Owner

No posted commitment with the hotkey in the title was found. Please commit on-chain using the exact PR head. For an already-open PR, use:

github-pr:unarbos/ninja#1582@44633c69a78ed5ae39435dce28454dfa11373161

For pre-PR protection, commit the head before opening the PR with:

github-pr-head:unarbos/ninja@44633c69a78ed5ae39435dce28454dfa11373161

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

notice: missing-commitment Validator notice that no matching on-chain PR commitment was found.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants