增加新浪财经热搜股票榜

[yichuanzhao99-ctrl]

Description

Add sinafinance stock-rank command - 新浪财经热搜榜

新增 stock-rank 命令，支持抓取新浪财经移动端首页股票热搜榜数据：

• 支持多个市场的股票热搜榜：A股(cn)、港股(hk)、美股(us)、外汇(wh)、期货(ft)

同时更新了 docs/adapters/browser/sinafinance.md 文档。

Type of Change

• ✨ New feature
• 📝 Documentation

Checklist

• Added doc page under docs/adapters/ (updated existing sinafinance.md)
• Used positional args for the command's primary subject (uses --market flag for market selection)

Screenshots / Output

# A股热搜榜
$ opencli sinafinance stock-rank
Column | Name    | Symbol  | Market | Price  | Change | Url
-------+---------+---------+--------+--------+--------+-----
1      | 贵州茅台 | 600519  | A股    | 1850.0 | +2.3%  | https://finance.sina.com.cn/...

# 港股热搜榜
$ opencli sinafinance stock-rank --market hk

# 期货热搜榜
$ opencli sinafinance stock-rank --market ft

# 外汇热搜榜
$ opencli sinafinance stock-rank --market wh

# 美股热搜榜
$ opencli sinafinance stock-rank --market us

[jackwener]

Review

Good feature direction — stock ranking data from Sina Finance is useful. A few issues need fixing before merge:

1. Security: String interpolation injection in evaluate

const marketType = '${market}';

User input is directly interpolated into the evaluate string. Use JSON.stringify:

const marketType = ${JSON.stringify(market)};

2. Missing market validation

Add choices to the arg definition to restrict valid values:

{ name: 'market', type: 'string', default: 'cn', choices: ['cn', 'hk', 'us', 'wh', 'ft'], help: '...' }

3. Column names inconsistent with other sinafinance commands

Other commands use lowercase (title, date, url). This PR uses PascalCase (Column, Name, Price). Please use lowercase: rank, name, symbol, market, price, change, url.

4. Unrelated change in rolling-news.ts

The ?from=opencli URL tracking change is unrelated to stock-rank and modifies existing output. Please remove it from this PR.

5. Null safety on tabEl

const tabEl = document.querySelector('[data-type="' + marketType + '"]');
const marketName = tabEl.textContent; // tabEl may be null

Use optional chaining: tabEl?.textContent || marketType

6. Add navigateBefore: false

The func already calls page.goto(), so the framework's pre-navigation is redundant. Add navigateBefore: false to avoid double navigation.

[jackwener]

All review issues fixed. LGTM.