CLI: reject invalid player seek and volume values before connecting
还没有人认领这个 Issue。
- #90 来自 @bfs2021 —— 已关闭，未合并
评估
调研方向
从 spotapi player seek 和 spotapi player volume 子命令的 CLI 参数解析开始，阅读 Player.seek() 和 Player.volume() 中现有的范围检查，以保持相同的上下限。添加小型 argparse 类型校验器，拒绝负数的 seek 值以及不在 0-100 范围内的音量，并以退出状态 2 退出，同时保持 0 和 100 有效。当离线测试覆盖负数 seek、低于 0 的音量、高于 100 的音量以及两个边界值时即完成；该 issue 要求在开始前先评论，以避免重复工作。
由索引模型根据 Issue 内容生成。
描述
Problem
The Python player API correctly rejects negative seek positions and volume values outside 0–100. The CLI currently parses both as unrestricted integers, so obviously invalid input can reach account/session setup before producing an error.
Proposed change
Validate these values during argument parsing:
spotapi player seek MILLISECONDS: integer greater than or equal to zero.spotapi player volume PERCENT: integer from zero through 100.
Acceptance criteria
- Invalid values fail through
argparsewith a clear range message and exit status 2. - Invalid input is rejected before reading account state or opening a Spotify connection.
- Boundary values
0and100remain valid where applicable. - Existing valid commands retain their current behavior.
- Offline tests cover negative seek, volume below zero, volume above 100, and boundaries.
Scope
Keep validation helpers small and local to the CLI. The existing validation inside Player.seek() and Player.volume() must remain as the library-level safety boundary.
Comment before starting so duplicate work can be avoided.
- 主要语言
- Python
- 星标
- 493
- 派生
- 40
- 平均合并
- 5 分钟
- 30 天内合并 PR
- 1
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Aran404/SpotAPI 的其他 Issue
-
CLI: expose public search language with --language可能已有人在做 @semajzandrews 今天认领。 未关闭enhancement good first issue
难度 2/5 1-3 小时 新手友好度 62/100
-
documentation enhancement help wanted
难度 4/5 3-5 天 新手友好度 42/100
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 66/100
deepchem/deepchem#5289 · 1 条评论 ·
维护者通常 2 天内回复
-
难度 1/5 1 小时以内 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 60/100
UKGovernmentBEIS/inspect_evals#2892 · 1 条评论 ·
维护者通常 1 天内回复
-
Versioning docs: say that no output from the 0.46 warning check means nothing is affected可能已有人在做 @grelinfo 今天认领。 未关闭documentation priority:low size:S
难度 1/5 1 小时以内 新手友好度 72/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 88/100
alphacrack/readme2demo#293 ·