Skip to content

fix: handle formatter paths and failed login flow - #9

Merged
WardLu merged 2 commits into
mainfrom
codex/fix/issue-7-source-path
Aug 3, 2026
Merged

fix: handle formatter paths and failed login flow#9
WardLu merged 2 commits into
mainfrom
codex/fix/issue-7-source-path

Conversation

@WardLu

@WardLu WardLu commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Fixes #7
Fixes #8

变更

  • 为格式化工具增加显式源目录/目标目录参数和源目录校验。
  • 登录失败时停止后续目录扫描,避免继续请求 None/ks/...
  • 校验知识库 API 地址,拒绝无效地址并保留默认 API 地址回退。
  • 为登录失败、无效 API 地址和默认 API 地址回退增加回归测试。
  • 更新 README 中的故障排查说明。

验证

  • python3 -m pytest --ignore=app -q:7 passed
  • PYTHONPYCACHEPREFIX=/tmp/issue78-pycache python3 -m py_compile ...:通过
  • git diff --check:通过

现有 app 工作区修改未包含在本 PR 中。

@ecc-tools

ecc-tools Bot commented Aug 3, 2026

Copy link
Copy Markdown

Analyzing 200 commits...

@ecc-tools

ecc-tools Bot commented Aug 3, 2026

Copy link
Copy Markdown

Analysis Complete

Generated ECC bundle from 2 commits | Confidence: 50%

View Pull Request #10

Repository Profile
Attribute Value
Language Python
Framework Not detected
Commit Convention conventional
Test Directory separate
Changed Files (5)
Metric Value
Files changed 5
Additions 292
Deletions 40

Top hotspots

Path Status +/-
tests/test_wiznote_downloader.py added +102 / -0
tools/obsidian_formatter.py modified +51 / -33
tests/test_obsidian_formatter.py added +78 / -0
tools/wiznote_downloader.py modified +39 / -3
README.md modified +22 / -4

Top directories

Directory Files Total changes
tests 2 180
tools 2 126
. 1 26
Analysis Depth Readiness (commit-history, 7%)

ECC Tools uses this to decide whether recommendations should stay at commit-history/setup guidance or expand into CI, security, harness, reference-set, AI-routing, and team backlog work.

Area Status Evidence / Next Step
Commit history Partial 2 commits sampled
CI/CD signals Missing Add workflow files or CI troubleshooting evidence so ECC Tools can reason about pipeline setup.
Security evidence Missing Add AgentShield, audit, SARIF, SBOM, or security review evidence so recommendations can cover security posture.
Harness configuration Missing Add Claude, Codex, OpenCode, Zed, dmux, MCP, plugin, or cross-harness config evidence for harness-agnostic recommendations.
Reference/eval evidence Missing Add fixtures, golden traces, reference sets, or evaluator benchmarks so deeper recommendations have regression evidence.
AI routing and cost controls Missing Add model-routing, budget, usage, or cost-control files before relying on AI-heavy automation recommendations.
Team handoff and project tracking Missing Add roadmap, runbook, project, Linear, or follow-up tracking docs so generated work can land in a team queue.
Reference Set Readiness (0/7, 0%)
Area Status Evidence / Next Step
Deep analyzer corpus Missing Add analyzer fixture, golden, benchmark, or reference-set files that can catch analyzer regressions.
RAG/evaluator comparison Missing Add retrieval or evaluator reference-set comparison fixtures with expected ranking behavior.
PR salvage/review corpus Missing Add stale-PR, review-thread, reopen-flow, or salvage reference cases for queue cleanup automation.
Discussion triage corpus Missing Add public discussion triage fixtures, golden cases, or reference sets for informational, answered, and no-response classifications.
Harness compatibility Missing Add cross-harness, adapter-compliance, or harness-audit evidence for Claude, Codex, OpenCode, Zed, dmux, and agent surfaces.
Security evidence Missing Attach security evidence such as SBOMs, SARIF, audit reports, or AgentShield evidence packs.
CI failure-mode evidence Missing Add captured CI failure logs, dry-run fixtures, or troubleshooting docs for common workflow failure modes.
Generated Instincts (6)
Domain Count
git 2
code-style 3
testing 1

After merging, import with:

/instinct-import .claude/homunculus/instincts/inherited/wiznote-to-obsidian-instincts.yaml

Files

  • .claude/ecc-tools.json
  • .claude/skills/wiznote-to-obsidian/SKILL.md
  • .agents/skills/wiznote-to-obsidian/SKILL.md
  • .agents/skills/wiznote-to-obsidian/agents/openai.yaml
  • .claude/identity.json
  • .codex/config.toml
  • .codex/AGENTS.md
  • .codex/agents/explorer.toml
  • .codex/agents/reviewer.toml
  • .codex/agents/docs-researcher.toml
  • .claude/homunculus/instincts/inherited/wiznote-to-obsidian-instincts.yaml

ECC Tools | Everything Claude Code

@WardLu
WardLu merged commit 54f4bc5 into main Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant