Skip to content

maint: port verified community fixes onto current main#627

Merged
tirth8205 merged 5 commits into
mainfrom
codex/reconcile-port-batch
Jul 17, 2026
Merged

maint: port verified community fixes onto current main#627
tirth8205 merged 5 commits into
mainfrom
codex/reconcile-port-batch

Conversation

@tirth8205

Copy link
Copy Markdown
Owner

Summary

Ports four verified community fixes onto current main without restoring their stale/conflicting branch history:

Each source contributor is retained through a Co-Authored-By trailer on the corresponding commit.

Verification

  • Full suite: 1,476 passed, 2 xpassed
  • Incremental/ignore suite: 62 passed
  • Skills/install suite: 169 passed
  • Documentation regressions: 2 passed
  • Ruff on every changed Python file: clean
  • git diff --check: clean

The locally installed repo-wide Ruff version also reports 44 pre-existing findings in untouched files; this branch introduces none and cleans the two touched test modules.

Fixes #91.
Fixes #561.

Source contributions: #92, #530, #562, #625.

tirth8205 and others added 4 commits July 17, 2026 14:23
Port the final reviewed safe-anywhere versus root-relative behavior from
PR #92 on top of PR #583's trailing-slash support. Keep unanchored custom
directory patterns gitignore-compatible while anchoring ambiguous defaults.

Co-Authored-By: Chintan Gohil <echintangohil@gmail.com>
Port PR #530 onto the current hardened config merger. Prefer
opencode.jsonc/opencode.json, write a top-level mcp local command with an
explicit --repo argument, and warn without overwriting the legacy dotfile.

Co-Authored-By: fdcp <fdcp@163.com>
Port PR #562 onto current main and include the bundled review-changes skill
that the stale source diff omitted. Add one consistency regression covering
all generated and bundled skill frontmatter.

Co-Authored-By: kankunnawat <kan.k@bitazza.com>
Carry PR #625's zsh fix through all current user docs, using double quotes
instead of single quotes so the examples also work in Windows cmd.exe. Add
a regression that prevents unquoted extras from returning.

Co-Authored-By: nyxst4ck <289980115+nyxst4ck@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

code-review-graph review

Overall risk: 0.55 (MEDIUM) — 27 changed function(s)/class(es), 0 affected flow(s), 11 test gap(s)

Risk-scored changes

Risk Level Symbol Location Tested
0.55 medium code_review_graph/skills.py::install_platform_configs code_review_graph/skills.py:404 no
0.40 medium code_review_graph/incremental.py::_load_ignore_patterns code_review_graph/incremental.py:361 no
0.40 medium code_review_graph/incremental.py::_should_ignore code_review_graph/incremental.py:388 no
0.35 low code_review_graph/skills.py::_build_server_entry code_review_graph/skills.py:241 no
0.35 low code_review_graph/skills.py::_warn_legacy_opencode_config code_review_graph/skills.py:261 no
0.30 low code_review_graph/skills.py::_opencode_config_path code_review_graph/skills.py:35 no
0.30 low tests/test_documentation.py::test_pip_extra_examples_use_cross_shell_double_quotes tests/test_documentation.py:31 (test)
0.30 low tests/test_documentation.py::test_current_user_docs_have_no_unquoted_pip_extras tests/test_documentation.py:40 (test)
0.30 low tests/test_incremental.py::TestDataDirRegistry tests/test_incremental.py:376 no
0.30 low tests/test_incremental.py::TestDataDirRegistry.test_registry_fallback_to_env_var tests/test_incremental.py:420 (test)

Test gaps

  • code_review_graph/incremental.py::_load_ignore_patterns (code_review_graph/incremental.py:361)
  • code_review_graph/incremental.py::_should_ignore (code_review_graph/incremental.py:388)
  • code_review_graph/skills.py::_opencode_config_path (code_review_graph/skills.py:35)
  • code_review_graph/skills.py::_build_server_entry (code_review_graph/skills.py:241)
  • code_review_graph/skills.py::_warn_legacy_opencode_config (code_review_graph/skills.py:261)
  • ...and 6 more without direct tests

Token savings: this graph-backed report used ~99,823 fewer tokens (~95%) than reading every changed file in full (estimated, chars/4 approximation).


Powered by code-review-graph — local-first analysis; no code leaves the CI runner.

@tirth8205
tirth8205 merged commit ceb6fba into main Jul 17, 2026
10 checks passed
@tirth8205
tirth8205 deleted the codex/reconcile-port-batch branch July 17, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill name: fields use Title Case instead of lowercase-hyphen convention Nested dependency dirs not ignored + missing framework patterns in defaults

1 participant