JS修正してGoにコントリビュートした

Treasure Advent Calendar 2019 の22日目の記事です。

adventar.org

先日、Treasureでもお世話になったGoへコントリビュートしました！

Goへのコントリビュート成功しました
わーい🎉https://t.co/V7VGUJDBNW
— ramenjuniti (@ramenjuniti) 2019年11月19日

この記事では、初のOSSへコントリビュート成功ということで、その経緯を書いていこうと思います

ある日

Go Conference 2019 Autumnに参加した影響か、Goへの思いが高まりGoへコントリビュートしたいなーと思いながらGithubでGoのコードを覗いてました

気になるものを見つける

こんなのあったんかhttps://t.co/I1ebjwI5M7
— ramenjuniti (@ramenjuniti) 2019年11月17日

どうやらGoにはgophertoolというChrome 拡張機能があるらしい

粗探しをする

中身を覗いてみると5~9年前で更新が止まってるっぽい

f:id:ramenjuniti:20191219162621p:plain — コントリビュートする前

なんか怪しそう！

過去にChrome拡張を作ったことがあり雰囲気でコードは読むことができたので、何かミスってるとこないかなーと粗探しをしてみることに

お！？

Chrome拡張のAPIドキュメントを見ながら、gophertool/background.jsを粗探ししてると

chrome.omnibox.onInputEntered.addListener(function(t) {
  var url = urlForInput(t);
  if (url) {
    chrome.tabs.getSelected(null, function(tab) { // <-- お？
      if (!tab) return;
      chrome.tabs.update(tab.id, { "url": url, "selected": true });
    });
  }
});

「chrome.tabs.getSelectedを使ってるんか、ドキュメント見てみるか」

f:id:ramenjuniti:20191219171948p:plain

「ん？」

f:id:ramenjuniti:20191219174346p:plain

「うおー！Deprecatedやんけー！」

興奮しました

「修正も難しくなさそうだし、プルリク作ったらマージしてもらえそう！」

興奮しました

プルリクを投げる

修正できそうなコードを見つけることができたので、実際にプルリクを投げてみることに

Goにはコントリビュートの手順に関するドキュメントが用意されています golang.org

ドキュメントに従い、issueを作り github.com

GoではレビューシステムにGerritというシステムを利用しているので、Gerrit上でコードを以下のように修正してプルリクを投げました

chrome.omnibox.onInputEntered.addListener(function(t) {
  var url = urlForInput(t);
  if (url) {
    chrome.tabs.query({ "active": true, "currentWindow": true }, function(tab) { // <- 修正箇所
      if (!tab) return;
      chrome.tabs.update(tab.id, { "url": url, "selected": true });
    });
  }
});