docs(readme): record daemon-strict migration completion - #14
Merged
Conversation
jphein
added a commit
that referenced
this pull request
May 8, 2026
ruff F401: `pytest` imported but unused. The file uses no `pytest.` calls — fixtures (tmp_path, monkeypatch) are pytest's implicit injections, not pytest module references. Pre-existing on main since 41359ba (2026-05-07, row 33's daemon routing tests). Surfaced now because PR #15 is the first PR- triggered CI run that touches the lint job since the import landed. Same fix applies to PR #14's lint failure once #15 merges into main and #14 rebases. 15/15 daemon tests still pass.
jphein
added a commit
that referenced
this pull request
May 8, 2026
Pre-existing windows-only failures since rows 32 and 34 landed (2026-05-05 and 2026-05-07). Surfaced now because PR #15 is the first PR-triggered Windows CI run that exercised these. 1. tests/test_cli_daemon.py::test_routes_projects_mode_to_daemon `endswith("/home/u/proj")` failed because Path.expanduser() .resolve() produces "D:\\home\\u\\proj" on Windows. Normalize separators before the suffix check. 2. tests/test_normalize.py::test_config_reads_from_hooks_block monkeypatch.setenv("HOME", ...) doesn't redirect Path.home() on Windows (Path.home consults USERPROFILE on Windows, HOME on POSIX). Set both so the test redirect works on both platforms. Both files PASS on Linux (no regression). Same fix would apply to PR #14 once #15 merges into main.
… table Adds the May 7 daemon-routing slice (commits 41359ba + 22ef562) to README's "Fork-ahead — open or pending" table so the inventory matches CLAUDE.md rows 33+34 and the fork-changes.yaml entries already committed in 848b1dc. Single consolidated row covering both the mcp_server.py and cli.py changes since they ship together as one migration. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
jphein
force-pushed
the
docs/daemon-routing-followups
branch
from
May 8, 2026 03:51
8d88dc4 to
86c4785
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Context
The migration shipped across two code commits:
docs/fork-changes.yaml and FORK_CHANGELOG.md were updated for both in 848b1dc. This PR catches up the README table only.
Test plan
🤖 Generated with Claude Code