Skip to content

feat(signature): the change_signature operation (#1533, 4/9) - #2156

Closed
vitali87 wants to merge 1 commit into
pr-split/split-1912-change-signature/pr-3from
pr-split/split-1912-change-signature/pr-4
Closed

vitali87 wants to merge 1 commit into
pr-split/split-1912-change-signature/pr-3from
pr-split/split-1912-change-signature/pr-4

Conversation

@vitali87

Copy link
Copy Markdown
Owner

Part 4 of the #1912 split (issue #1533). signature.py: SignatureChanger plans the change over the override hierarchy and every graph-known call site, stages it in an edit transaction, applies it, and enforces the contract when a re-ingest is available. change_signature() is the entry point.

Files changed

  • codebase_rag/editing/signature.py

Diff stats

+479 additions, -0 deletions (479 LOC)

Dependencies

This PR depends on: pr-3

Dependency graph

Merge in this order:

pr-1: feat(signature): contract expectation, shared call-site helpers and constants for change_signature (#1533, 1/9)
└── pr-2: feat(signature): parameter specs, mapping sources and the change_signature contract check (#1533, 2/9)
    └── pr-3: feat(signature): read the header, bind call arguments, render the rewrite (#1533, 3/9)
        └── pr-4: feat(signature): the change_signature operation (#1533, 4/9)  <-- this PR
            └── pr-5: feat(signature): expose change_signature through the CLI and MCP, with docs (#1533, 5/9)
                └── pr-6: test(signature): acceptance, transaction and MCP tests for change_signature (#1533, 6/9)
                    └── pr-7: test(signature): mapping tests for change_signature (#1533, 7/9)
                        └── pr-8: test(signature): definition and renamed-parameter tests for change_signature (#1533, 8/9)
                            └── pr-9: test(signature): hierarchy, receiver and default guards for change_signature (#1533, 9/9)

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)
  • ^feat/.*
  • ^fix/.*

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: vitali87/code-graph-rag/.coderabbit.yaml

Review profile: QUIET

Plan: Advanced

Run ID: d783ffcb-e60c-464f-bd39-f6740bd39f55

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@vitali87 vitali87 closed this Sep 22, 2026
@vitali87
vitali87 deleted the pr-split/split-1912-change-signature/pr-4 branch September 22, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant