Skip to content

Fix host:port interpolation in network latency test#7

Merged
Pnwcomputers merged 1 commit intomainfrom
codex/fix-variable-reference-errors-in-script
Oct 31, 2025
Merged

Fix host:port interpolation in network latency test#7
Pnwcomputers merged 1 commit intomainfrom
codex/fix-variable-reference-errors-in-script

Conversation

@Pnwcomputers
Copy link
Owner

@Pnwcomputers Pnwcomputers commented Oct 31, 2025

Summary

  • ensure host and port strings in the network latency test are constructed safely when strict mode is enabled
  • format the PsPing argument using -f to avoid PowerShell parsing errors

Testing

  • not run

https://chatgpt.com/codex/tasks/task_e_690508791e14832587fcb13d65040b88

@Pnwcomputers Pnwcomputers merged commit 2c4c029 into main Oct 31, 2025
3 checks passed
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
VS AI Update (#8)
ef7cc2e 
* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error
@Pnwcomputers Pnwcomputers mentioned this pull request Oct 31, 2025
Merged
9 tasks
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
VS AI Update (#8) (#9)
4095a0b 
* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
VS AI Update (#8) (#10)
f79df76 
* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
VS AI Update (#8) (#11)
dde4070 
* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
Version2.2 (#12)
e44f3b5 
* VS AI Update (#8)

* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error

* fix: address VS warnings

- Rename Run-Tool -> Invoke-Tool
- Replace use of  with
- Use Convert-ToolOutput in tool runner
- Keep -Args support via [Alias('Args')]
Pnwcomputers added a commit that referenced this pull request Oct 31, 2025
@Pnwcomputers
Version2.2 (#13)
1bd69a2 
* VS AI Update (#8)

* Update SystemTester.bat

* Update SystemTester.ps1

* Update SystemTester.bat

* Update README.md

* Update README.md

* Update README.md

* Update SystemTester.bat

* Update SystemTester.ps1

* Update README.md

* Create .gitkeep

* Create CODE_OF_CONDUCT.md

* Create CONTRIBUTING.md

* Create PULL_REQUEST_TEMPLATE.md

* Update CONTRIBUTING.md

* Create bug_report.md

* Create config.yml

* Create deployment_issue.md

* Create feature_request.md

* Create implementation-question.md

* Create COMPATABILITY.md

* Create INSTALLATION.md

* Create TROUBLESHOOTING.md

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

* Delete CONTRIBUTING.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Handle both script names in batch launcher (#3)

* Add network speed and update visibility to reports (#4)

* Improve GPU detection and batch reliability (#5)

* Handle tool verification exit statuses (#6)

* Update README.md

* Fix host:port string interpolation in latency test (#7)

* refactor: use approved verbs and fix analyzer issues

- Clean-ToolOutput -> Convert-ToolOutput (+call site)
- Run-Tool -> Invoke-Tool (+call sites)
- Generate-Report -> New-Report (+call sites)
- Replace  param with  (alias Args)
- Avoid  automatic var in PsPing
- Fix  comparison order
- Remove unused  var
- Repair GPU submenu strings causing parse error

* fix: address VS warnings

- Rename Run-Tool -> Invoke-Tool
- Replace use of  with
- Use Convert-ToolOutput in tool runner
- Keep -Args support via [Alias('Args')]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

codex

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Pnwcomputers