Add no-pr argument to CF LSP

[Saga4]

PR Type

Enhancement

---

Description

• Disable PR creation in LSP server initialization

---

Changes walkthrough 📝

	Relevant files
Configuration changes	server.py Add no_pr flag to LSP server init                                                codeflash/lsp/server.py Set args.no_pr = True before optimizer initialization +1/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Argument Definition Ensure that no_pr is declared in the CLI argument parser, otherwise assigning to this attribute will raise an AttributeError during initialization. args.no_pr = True # LSP server should not create PRs Processing Order Setting args.no_pr before process_pyproject_config may allow the configuration to overwrite this flag. Consider moving this assignment after processing the pyproject config to guarantee it remains true. args.no_pr = True # LSP server should not create PRs args = process_pyproject_config(args)

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Preserve no_pr after config processing Move this assignment below the call to process_pyproject_config so that any defaults or overrides loaded from the pyproject config cannot overwrite the no_pr flag. codeflash/lsp/server.py [55] +args = process_pyproject_config(args) args.no_pr = True # LSP server should not create PRs +self.optimizer = Optimizer(args) Suggestion importance[1-10]: 7 __ Why: Moving args.no_pr = True below the config processing ensures that any defaults from process_pyproject_config won’t overwrite the intended LSP server behavior, which is a relevant correctness improvement.	Medium
Possible issue	Validate no_pr attribute existence Guard the assignment with an hasattr check to ensure no_pr is a valid attribute on args and avoid silent attribute errors. codeflash/lsp/server.py [55] -args.no_pr = True # LSP server should not create PRs +if hasattr(args, "no_pr"): + args.no_pr = True # LSP server should not create PRs +else: + raise AttributeError("no_pr argument missing on args") Suggestion importance[1-10]: 3 __ Why: Adding an hasattr guard is a minor defensive check that’s likely unnecessary since args should include no_pr from the CLI parser, making this low-impact.	Low