Skip to content

chore: remove dead helpers in dspy/propose/utils - #130

Open
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-dead-helpers-in-dspy-propose-utils-0984ed
Open

detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-dead-helpers-in-dspy-propose-utils-0984ed

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 16, 2026

Copy link
Copy Markdown

Remove three unused top-level functions from dspy/propose/utils.py: create_instruction_set_history_string, parse_list_of_instructions, and get_program_instruction_set_string. Also drop the now-unneeded "import json" that only parse_list_of_instructions used.

These functions had no production callers, no test references, and were not re-exported in dspy/propose/init.py. They were added in the initial MIPRO release commit and never wired up; get_program_instruction_set_string was only called by the dead create_instruction_set_history_string. A repo-wide search confirms no imports or call sites.

History

Last materially changed June 2024 (stanfordnlp#1169 @XenonMolecule).


Dead Code PRs can be configured here.

@greptile-apps

greptile-apps Bot commented Sep 16, 2026

Copy link
Copy Markdown

Greptile Summary

This PR removes three isolated, unused instruction-history helpers from dspy/propose/utils.py and drops their now-unused json import.

  • Preserves all currently referenced proposal utilities.
  • Does not alter the exported dspy.propose API.
  • Leaves no remaining references to the removed helpers.

Confidence Score: 5/5

The PR appears safe to merge because it only removes isolated dead code with no tracked callers or public exports.

No actionable failures remain; repository references and package exports confirm that the deleted helpers and import were unused.

Important Files Changed

Filename Overview
dspy/propose/utils.py Removes three unreferenced helper functions and the json import used exclusively by one of them.

Reviews (1): Last reviewed commit: "chore: remove dead helpers in dspy/propo..." | Re-trigger Greptile

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.

0 participants