Skip to content

refactor: remove unused configuration alias helper - #147

Merged
anycodes merged 2 commits into
mainfrom
refactor/dead-code-cleanup-20261027-recovered
Oct 27, 2026
Merged

anycodes merged 2 commits into
mainfrom
refactor/dead-code-cleanup-20261027-recovered

Conversation

@anycodes

@anycodes anycodes commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Description

Remove the uncalled private _a() alias-construction wrapper. Public aliases, schema, package versions and server implementation are preserved. Fixes #146.

Type of Change

  • Refactoring
  • Documentation

Changes

  • Reuse existing behavior coverage for deletion-only maintenance.
  • Add bilingual architecture guidance, Unreleased changelog, decision and verification evidence.
  • Record preserved compatibility arguments, generator semantics and lifecycle candidates separately.

Testing

  • Latest-head CI passed: full Python 3.10/3.11/3.12 client/server matrix, public contract, runtime fixtures and Node provider engine; Docs links passed.
  • Local lint, formatting, mypy, both distribution builds and clean installation checks passed. Client tests: 2,345 passed / 26 skipped / 10 deselected.
  • Two local raw-HTTP server tests failed under altered sandbox forwarding; an independent HTTPServer negative control reproduced that behavior. The unchanged assertions pass in the remote matrix.
  • Real-cloud acceptance: 130 passing cases across the initial run and corrected single-case CLI invocation; 11 optional/conditional skips and 2 slow-case deselections remain explicit.
  • Run-scoped janitor dry run and subsequent independent inventory: no sandboxes remained. No API-key preparation or renewal was attempted.

Checklist / Definition of Done

  • Required current-head CI and remote unit matrix pass
  • Real-component acceptance and independent cleanup recorded
  • Proposed decision and evidence indexed under .agents/
  • English/Chinese docs and Unreleased changelog updated
  • Repository health: 106 pass / 1 warning / 0 failures
  • No sensitive information included

Verification records exact scope, commands, initial failures and limitations.

@anycodes
anycodes marked this pull request as ready for review October 27, 2026 09:03
@anycodes
anycodes merged commit c7a6635 into main Oct 27, 2026
13 checks passed
@anycodes

Copy link
Copy Markdown
Contributor Author

Verified actual squash merge after all current-head workflow runs and jobs passed. Merge commit: c7a6635. CI: https://github.com/Alibaba-Agent-Sandbox/python-sdk/actions/runs/40477035278, https://github.com/Alibaba-Agent-Sandbox/python-sdk/actions/runs/40477035155. Verification details and initial failures/skips remain recorded in the PR evidence file. No manual release or deployment was performed.

@anycodes anycodes mentioned this pull request Oct 27, 2026
5 tasks done
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.

Remove unused private configuration alias helper

1 participant