Web.AsyncHTTP: Added Web.AsyncHTTP. by mikoto2000 · Pull Request #847 · vim-jp/vital.vim

mikoto2000 · 2025-12-21T00:18:07Z

close #842

doc/vital/Web/AsyncHTTP.txt

mikoto2000 · 2025-12-23T14:03:03Z

Web.HTTP, Web.AsyncHTTP, Web.HTTP.Core に分割していこう、 Vitalize に失敗するようになっていたのでドラフトに戻しました。

mikoto2000 · 2025-12-23T14:47:29Z

Vitalize に失敗する問題を修正したのでドラフトを解除しました。

tsuyoshicho

@mikoto2000
確認できた改善点をコメントしました。
一部は修正必須と思うので(といってもtypo) Request changes にしました。

tsuyoshicho · 2025-12-24T01:47:39Z

doc/vital/Web/AsyncHTTP.txt

+	This function requires one of the clients, "curl" or "wget".
+
+	Example: >
+	let s:AsyncHTP = vital#{plugin-name}#new().import('Web.AsyncHTTP')


Suggested change

let s:AsyncHTP = vital#{plugin-name}#new().import('Web.AsyncHTTP')

let s:AsyncHTTP = vital#{plugin-name}#new().import('Web.AsyncHTTP')

typo fix

tsuyoshicho · 2025-12-24T01:50:32Z

doc/vital/Web/AsyncHTTP.txt

+python					*Vital.Web.AsyncHTTP-client-python*
+	"python" as "python3" or "python2" auto select.
+	(High priority for "python3".)


Core で Python のファイル依存をはずしたりしてますが、こちらは利用できますか?

(CoreとPython利用の組み合せがどうなっているかを確認したい感じです)

もし、非同期との関係を考えて、サポートしないほうがいいのであれば、clientから外すのはアリと思いますので、記述を更新してもらえれば

client の実体は、 HTTP, AsyncHTTP それぞれに存在しています。 (同期・非同期で実装が変わるため、それぞれで定義)
そのため、 python への依存は HTTP, AsyncHTTP それぞれから行っています。
ですので、 Core から python への依存は無くても大丈夫です。

Linux でですが、以下コードもエラーなく動きます。

let s:AsyncHTTP = vital#vital#import('Web.AsyncHTTP') function! s:user_cb(response) abort echomsg a:response " => Web.HTTP と同じ形式の辞書 endfunction call s:AsyncHTTP.request({ \ 'url': 'https://example.com', \ 'user_cb': function('s:user_cb'), \ 'clients': ['python2'], \ }) call s:AsyncHTTP.request({ \ 'url': 'https://example.com', \ 'user_cb': function('s:user_cb'), \ 'clients': ['python3'], \ })

tsuyoshicho · 2025-12-24T01:56:47Z

autoload/vital/__vital__/Web/AsyncHTTP.vim

+function! s:_vital_loaded(V) abort
+  let s:V = a:V
+  let s:Prelude = s:V.import('Prelude')
+  let s:HTTP = s:V.import('Web.HTTP')


この依存関係は削除して、利用している機能は Core に置き、それを HTTP/AsyncHTTP が利用するように改善したほうがいいかと

tsuyoshicho · 2025-12-24T01:59:08Z

autoload/vital/__vital__/Web/HTTP.vim

+    if has_key(s:clients.curl.errcode(), retcode)
+      throw 'vital: Web.HTTP: ' . s:clients.curl.errcode()[retcode]


せっかくのリファクタリングでもあるので

if s:clients.curl.errcode_has_key(retcode) throw 'vital: Web.HTTP: ' . s:clients.curl.errcode(retcode)

みたいなインタフェースにしてもいいかもしれませんね

errcode()が配列返すのがマストではないし、複数回利用するようなら、配列を初期化時にコピー相当でもいいので、

コピーする

関数IFにする

のどちらかに寄せたほうがいいかなと

autoload/vital/__vital__/Web/HTTP.vim

mikoto2000 · 2025-12-25T12:59:02Z

クライアントの指定方法を感知餓死していたため、 curl 以外の動作確認ができていない状態でした。
今正しい方法で curl 以外の動作確認をしたらエラーになる状態でしたので、またドラフトに戻します...

tsuyoshicho · 2025-12-25T23:36:44Z

doc/vital/Web/AsyncHTTP.txt

+parseHeader({headers})			*Vital.Web.HTTP.parseHeader()*
+	Parse {headers} list to a dictionary.
+	Duplicated fields are overwritten.
+
+encodeURI({param})			*Vital.Web.HTTP.encodeURI()*
+	Encode params as URI query.
+
+decodeURI({str})			*Vital.Web.HTTP.decodeURI()*
+	Decode string as URI params.
+
+encodeURIComponent({str})		*Vital.Web.HTTP.encodeURIComponent()*
+	Encode param as URI components.


ここらへん AsyncHTTP になってないため、タグ重複エラーしてるみたいです。

正式で出すにあたって、修正しておいてください。

….HTTP and Web.AsyncHTTP.

…TTP to Web.HTTP.Core.

…eb.HTTP and Web.AsyncHTTP to Web.HTTP.Core

mikoto2000 · 2025-12-26T23:41:13Z

リカバリ力が足りず、一から細かく刻みつつ再実装してしまいました。
せっかくレビューいただいたのに申し訳ありません...。
指摘は反映させたつもりです。

tsuyoshicho

見てみました

tsuyoshicho · 2025-12-27T02:13:35Z

doc/vital/Web/AsyncHTTP.txt

+CLIENT					*Vital.Web.AsyncHTTP-client*
+
+The following can be used.
+(TODO: More document.  Especially about limitation.)


これは削除していいと思います。

また、Python系について、Pythonコマンド+スクリプトファイル+引数を AsyncProcess するという方法はなくはないので、一応TODOとしてPython3対応は将来予定として記載しておくのはどうでしょう?

(というか、そろそろ Python2 はもういい気がするな...w)

tsuyoshicho · 2025-12-27T02:16:30Z

doc/vital/Web/AsyncHTTP.txt

+	"unixSocket"	Default: (None)
+		Use --unix-sokect (only curl >= 7.40.0)
+
+	"user_cb"	Default: (None)


ここだけスネークケースなのは微妙なので

Suggested change

"user_cb" Default: (None)

"userCallback" Default: (None)

という引数にするのはどうでしょうか?

(そもそも Vim の記法としてどうかはあるけど)

tsuyoshicho · 2025-12-27T02:18:18Z

autoload/vital/__vital__/Web/HTTP/Core.vim

+  return result
+endfunction
+
+" public interface implements


うえの build 系なども利用はあるので、そちらも public な気がします。

逆に Core 内に閉じる関数もなくない気がしますが... (あったら、やるなら s:_hoge 化?)

なお関数の記法がいろいろなのは、内部処理だったからでしょうね、そこはとりあえずそのままでもいいと思います。

…named according to the naming convention.

tsuyoshicho

おつかれさまです、安定したと思うので、詳細に見てみました。

ちょっと迷走しましたが、settings の client と、urlencode_char 、依存関係のところは対処が必要と思います。

autoload/vital/__vital__/Web/HTTP/Core.vim

tsuyoshicho · 2025-12-28T00:04:11Z

autoload/vital/__vital__/Web/HTTP/Core.vim

+  let settings = {
+  \   'method': 'GET',
+  \   'headers': {},
+  \   'client': ['python', 'curl', 'wget', 'python3', 'python2'],


クライアントは各実装依存となったので、こちらで固定で設定するのではなく

request で追加設定する

引数を追加して、引数から client 設定する

などのAPI設計にしたようが良いと思われます。

autoload/vital/__vital__/Web/HTTP/Core.vim

autoload/vital/__vital__/Web/AsyncHTTP.vim

autoload/vital/__vital__/Web/HTTP.vim

autoload/vital/__vital__/Web/HTTP/Core.vim

tsuyoshicho

追加指摘です。

一応helpは
https://github.com/vim-jp/vimdoc-ja-working/wiki/Guide
のフォーマッタ設定を使うといいのですが、現状ちゃんと書いてあるから問題はないといえばないです

やたら差分でますが、調整の範囲ではあるという...

tsuyoshicho · 2025-12-28T04:12:44Z

doc/vital/Web/AsyncHTTP.txt

+Maintainer: mattn <mattn.jp@gmail.com>
+            thinca <thinca+vim@gmail.com>


ここは自分にしてください
(元にしたのが Web.HTTP と書いてもいいかもです

Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>

mikoto2000 · 2025-12-28T11:49:07Z

一応helpは
https://github.com/vim-jp/vimdoc-ja-working/wiki/Guide
のフォーマッタ設定を使うといいのですが、現状ちゃんと書いてあるから問題はないといえばないです

やたら差分でますが、調整の範囲ではあるという...

82ad926 で対応いたしました。
Web.HTTP をベースに書いたのですが、あえて改行を入れている箇所以外はフォーマッタと同じ結果になるようでした。
AsyncHTTP としたせいで行幅を越していた 1 行を修正しています。

tsuyoshicho

たぶんいいとは思います

(Coreのテストは悩むところ)

to: @thinca @mattn
レビューおねがいします。
あるいは他にレビュー向きな人がいれば、そちらへ ping していただければ

(非同期だと lambdalisue さんとかだろうか...)

test/Web/AsyncHTTP.vimspec

Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>

ujihisa requested review from mattn and thinca December 21, 2025 00:20

tsuyoshicho reviewed Dec 21, 2025

View reviewed changes

doc/vital/Web/AsyncHTTP.txt Show resolved Hide resolved

mikoto2000 marked this pull request as draft December 23, 2025 14:02

mikoto2000 marked this pull request as ready for review December 23, 2025 14:46

tsuyoshicho requested changes Dec 24, 2025

View reviewed changes

mikoto2000 marked this pull request as draft December 25, 2025 12:59

tsuyoshicho requested changes Dec 25, 2025

View reviewed changes

mikoto2000 added 9 commits December 26, 2025 22:54

Web.AsyncHTTP: Added initial implements.

0d8ed31

Web.AsyncHTTP: Deleted python clients.

ed41ead

Web.AsyncHTTP: Move errcode to Web.HTTP.Core and share it between Web…

a894e7a

….HTTP and Web.AsyncHTTP.

Web.AsyncHTTP: Moved common functions between Web.HTTP and Web.AsyncH…

ebff999

…TTP to Web.HTTP.Core.

Web.AsyncHTTP: Moved common public function implementations between W…

ea76891

…eb.HTTP and Web.AsyncHTTP to Web.HTTP.Core

Web.AsyncHTTP: Added test.

4e03b15

Web.AsyncHTTP: Added documents.

a946315

Web.AsyncHTTP: Remove unused dependencies.

8f525ee

Web.AsyncHTTP: Fix typo.

43f7526

mikoto2000 force-pushed the add-async-http branch from 75a63a6 to d14a759 Compare December 26, 2025 23:32

Web.AsyncHTTP: Added example in document.

d14a759

mikoto2000 marked this pull request as ready for review December 26, 2025 23:41

tsuyoshicho reviewed Dec 27, 2025

View reviewed changes

mikoto2000 added 2 commits December 27, 2025 22:08

Web.AsyncHTTP: Deleted TODO.

26bc379

Web.AsyncHTTP: Added TODO about async python3.

408d7c2

mikoto2000 force-pushed the add-async-http branch from 8ad4e39 to 78617a8 Compare December 27, 2025 13:20

mikoto2000 added 2 commits December 27, 2025 22:20

Web.AsyncHTTP: Rename callback key to userCallback.

78617a8

Web.AsyncHTTP: Public and private interfaces have been grouped and re…

595f7f3

…named according to the naming convention.

tsuyoshicho requested changes Dec 28, 2025

View reviewed changes

mikoto2000 and others added 4 commits December 28, 2025 20:04

Web.AsyncHTTP: Added Web.HTTP.Core module in Web.AsyncHTTP.

1a61950

Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>

Web.AsyncHTTP: Deleted unused function.

47f4025

Web.AsyncHTTP: Fixed Maintainer.

0e30740

Web.AsyncHTTP: Formated document.

82ad926

tsuyoshicho approved these changes Dec 28, 2025

View reviewed changes

test/Web/AsyncHTTP.vimspec Outdated Show resolved Hide resolved

Web.AsyncHTTP: Deleted invalid empty line in AsyncHTTP.vimspec.

b56818d

Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>

	let s:AsyncHTP = vital#{plugin-name}#new().import('Web.AsyncHTTP')
	let s:AsyncHTTP = vital#{plugin-name}#new().import('Web.AsyncHTTP')

		if has_key(s:clients.curl.errcode(), retcode)
		throw 'vital: Web.HTTP: ' . s:clients.curl.errcode()[retcode]

		Maintainer: mattn <mattn.jp@gmail.com>
		thinca <thinca+vim@gmail.com>

Conversation

mikoto2000 commented Dec 21, 2025

Uh oh!

Uh oh!

mikoto2000 commented Dec 23, 2025

Uh oh!

mikoto2000 commented Dec 23, 2025

Uh oh!

tsuyoshicho left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mikoto2000 commented Dec 25, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mikoto2000 commented Dec 26, 2025

Uh oh!

tsuyoshicho left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tsuyoshicho left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tsuyoshicho left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mikoto2000 commented Dec 28, 2025

Uh oh!

tsuyoshicho left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

tsuyoshicho left a comment •

edited

Loading

tsuyoshicho left a comment •

edited

Loading