lgtm 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 58d58c1e75a0bf166829f769000c82541830873d
-  data.tar.gz: 09623157b73c40cdbdf24485cdceeba115cd9470
+  metadata.gz: 8ceb211c29e4f2f4cbc9137f9ab452f143e9a0ef
+  data.tar.gz: 1c3a9666fc68d7c3dfee08fd4d2623579e14c346
 SHA512:
-  metadata.gz: 613967c95a80b9b1a1c6b31291d93fe6081d07a06b764258dc7aa2e35b92db25088dcec6ccb10793e9eb633aa23836c82410d0f332e557de647951c3c46ce3ea
-  data.tar.gz: 0797524047fae01500c43a21de80ab484ef2fb2343851bbce7304221b3d1c8cad017e755845cb4cfabcede599c242aede7b0df9fa8491db09ac823d4d360460f
+  metadata.gz: ccff47d835f76dbe2629b58bea948124757b4c524e537ec5accb010fe380d47997957126a8b2518409c5200dff8bc1d6b9751e25ae0e1d4d265680dbdd2fc682
+  data.tar.gz: 47745e86a76bf814659f0abb209c4b7489fe6405ced8a333c07805ae1b91c67c00d3d1db9dcdb3bef1f4f5c54670a0f72893abbeac81978bfeba33cd4ce37a09

data/Rakefile

@@ -1,6 +1,6 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/console

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
-require "bundler/setup"
-require "lgtm"
+require 'bundler/setup'
+require 'lgtm'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +10,5 @@ require "lgtm"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start

data/exe/lgtm

@@ -1,6 +1,29 @@
 #! /usr/bin/env ruby
 
-require 'lgtm'
+require 'thor'
+require_relative '../lib/lgtm'
 
-print Lgtm::In.new.fetch_lgtm_link_markdown!
+class Cli < Thor
+  desc 'link NUMBER', 'get lgtm markdown format. you can set pick index default or 0 by random'
+  def link(picker = :random)
+    print Lgtm::In.new.fetch_lgtm_link_markdown!(:link, picker)
+  end
 
+  desc 'image NUMBER', 'get image url format. you can set pick index default or 0 by random'
+  def image(picker = :random)
+    print Lgtm::In.new.fetch_lgtm_link_markdown!(:image, picker)
+  end
+
+  desc 'double', 'output of stdout to link, stderr to image url'
+  def double(picker = :random)
+    STDOUT.print Lgtm::In.new.fetch_lgtm_link_markdown!(:link, picker)
+    STDERR.print Lgtm::In.new.fetch_lgtm_link_markdown!(:image, picker)
+  end
+
+  map 'l' => 'link'
+  map 'i' => 'image'
+  map 'img' => 'image'
+  map 'd' => 'double'
+end
+
+Cli.start(ARGV)

data/lgtm.gemspec

@@ -4,24 +4,25 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'lgtm/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "lgtm"
+  spec.name          = 'lgtm'
   spec.version       = Lgtm::VERSION
-  spec.authors       = ["HaiTo"]
-  spec.email         = ["kimura@sansan.com"]
+  spec.authors       = ['HaiTo']
+  spec.email         = ['kimura@sansan.com']
 
   spec.summary       = 'fetch image url in lgtm.in'
   spec.description   = 'fetch image url in lgtm.in to pbcopy'
   spec.homepage      = 'https://github.com/haito/lgtm'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.files         = `git ls-files -z`.split("\x0").reject {|f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir        = 'exe'
+  spec.executables   = spec.files.grep(%r{^exe/}) {|f| File.basename(f) }
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency "bundler", "~> 1.12"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency 'bundler', '~> 1.12'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'pry'
   spec.add_development_dependency 'pry-byebug'
   spec.add_dependency 'mechanize'
+  spec.add_dependency 'thor'
 end

data/lib/lgtm.rb

@@ -1,25 +1,31 @@
-require "lgtm/version"
+require 'lgtm/version'
 require 'mechanize'
 
 module Lgtm
+  class Fixnum
+    MAX = 4_611_686_018_427_387_903
+  end
+
   class In
     LGTM_LINK_BASE = 'http://lgtm.in/i'.freeze
-    NOT_SET_VALIAVLE_ALERT_1 = "please set enviromant variable LGTM_IN_USER_PATH \n"
-    NOT_SET_VALIAVLE_ALERT_2 = "lgtm.in mypage example ->  http://lgtm.in/l/HaiTo \n"
-    NOT_SET_VALIAVLE_ALERT_3 = "in .bashrc, .zshrc -> export LGTM_IN_USER_PATH='http://lgtm.in/l/HaiTo' \n"
+    NOT_SET_VALIAVLE_ALERT_1 = "please set enviromant variable LGTM_IN_USER_PATH \n".freeze
+    NOT_SET_VALIAVLE_ALERT_2 = "lgtm.in mypage example ->  http://lgtm.in/l/HaiTo \n".freeze
+    NOT_SET_VALIAVLE_ALERT_3 = "in .bashrc, .zshrc -> export LGTM_IN_USER_PATH='http://lgtm.in/l/HaiTo' \n".freeze
 
-    REFLASH_TIME = 604_800.freeze
+    REFLASH_TIME = 604_800
 
     LAST_EXECUTED_AT_TIME_PATH = 'last_executed_at.time'.freeze
     CACHED_LINKS_PATH = 'cached_links.links'.freeze
-    BASE_PATH = File.expand_path('..', __FILE__)
+    BASE_PATH = '/tmp'.freeze
 
-    LINK_PAR_PAGE_BASE = 32.freeze
+    LINK_PAR_PAGE_BASE = 32
 
-    LGTM_MARKDOWN = '[![LGTM](__LGTM_IMAGE_DETAILS_PATH__)](__LGTM_IMAGE_PATH__)'
+    LGTM_MARKDOWN = '[![LGTM](__LGTM_IMAGE_DETAILS_PATH__)](__LGTM_IMAGE_PATH__)'.freeze
     LGTM_MARKDOWN_DETAIL_PATH_BASE = 'http://lgtm.in/p/'.freeze
     LGTM_MARKDOWN_IMAGE_PATH_BASE = 'http://lgtm.in/i/'.freeze
-    
+
+    LgtmLinkStruct = Struct.new(:index, :link, :image)
+
     def not_set_valiavle_alert
       print NOT_SET_VALIAVLE_ALERT_1
       print NOT_SET_VALIAVLE_ALERT_2
@@ -37,24 +43,40 @@ module Lgtm
       end
     end
 
-    def fetch_lgtm_link_markdown!
-      links =
+    def fetch_lgtm_link_markdown!(mode, picker)
+      link_structs =
         if Time.now.to_i - @last_executed_at.to_i > REFLASH_TIME
           update_executed_at!
 
-          links = fetch_links
-          cache_links!(links)
+          link_structs = fetch_links
+          cache_links!(link_structs)
 
-          links
+          link_structs
         else
           @cached_links.nil? ? fetch_links : Marshal.load(@cached_links)
         end
 
-      link = links.sample
+      link_struct =
+        case picker.to_i
+        when 1..Fixnum::MAX
+          link_structs.compact[picker.to_i]
+        else
+          link_structs.compact.sample
+        end
+
+      unless link_struct
+        print "you could set to NUMBER at between 0 and #{link_structs.size}"
+        exit(1)
+      end
 
-      hash = link.split('/').last
+      case mode
+      when :link
+        link_hash = link_struct.link.split('/').last
 
-      LGTM_MARKDOWN.gsub('__LGTM_IMAGE_DETAILS_PATH__', "#{LGTM_MARKDOWN_DETAIL_PATH_BASE + hash}").gsub('__LGTM_IMAGE_PATH__', "#{LGTM_MARKDOWN_IMAGE_PATH_BASE + hash}")
+        LGTM_MARKDOWN.gsub('__LGTM_IMAGE_DETAILS_PATH__', (LGTM_MARKDOWN_DETAIL_PATH_BASE + link_hash).to_s).gsub('__LGTM_IMAGE_PATH__', (LGTM_MARKDOWN_IMAGE_PATH_BASE + link_hash).to_s)
+      when :image
+        link_struct.image
+      end
     end
 
     private
@@ -69,7 +91,10 @@ module Lgtm
 
     def fetch_link_per_page!(cursor, links, agent)
       page = agent.get(@lgtm_my_page + "?page=#{cursor}")
-      links << page.links.map(&:href).select {|link| link.match(LGTM_LINK_BASE) }
+
+      links << page.links.select {|link| link.href.match(LGTM_LINK_BASE) }.map.with_index(links.size + 1) {|link, index|
+        LgtmLinkStruct.new(index, link.href, link.node.children[1].attributes['src'].value)
+      }
 
       if links.flatten!.size % LINK_PAR_PAGE_BASE == 0
         fetch_link_per_page!(cursor + 1, links, agent)
@@ -85,5 +110,11 @@ module Lgtm
     def cache_links!(links)
       File.open(BASE_PATH + '/' + CACHED_LINKS_PATH, 'w') {|f| f.write(Marshal.dump(links)) }
     end
+
+    def load_cached_links!
+      Marshal.load(@cached_links)
+    rescue
+      fetch_links
+    end
   end
 end

data/lib/lgtm/version.rb

@@ -1,3 +1,3 @@
 module Lgtm
-  VERSION = '0.4.0'
+  VERSION = '0.5.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lgtm
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - HaiTo
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-07-16 00:00:00.000000000 Z
+date: 2016-08-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -94,6 +94,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: thor
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: fetch image url in lgtm.in to pbcopy
 email:
 - kimura@sansan.com
@@ -110,7 +124,6 @@ files:
 - README.md
 - Rakefile
 - bin/console
-- bin/lgtm
 - bin/setup
 - exe/lgtm
 - lgtm.gemspec

data/bin/lgtm

@@ -1,57 +0,0 @@
-#! /usr/bin/env ruby
-
-require 'mechanize'
-
-LGTM_LINK_BASE = 'http://lgtm.in/i'.freeze
-LGTM_HAITO_PAGE = ENV['LGTM_IN_USER_PATH']
-
-REFLASH_TIME = 604_800.freeze
-
-LAST_EXECUTED_AT_TIME_PATH = 'last_executed_at.time'.freeze
-CACHED_LINKS_PATH = 'cached_links.links'.freeze
-BASE_PATH = File.expand_path('..', __FILE__)
-
-LGTM_MARKDOWN = '[![LGTM](__LGTM_IMAGE_DETAILS_PATH__)](__LGTM_IMAGE_PATH__)'
-LGTM_MARKDOWN_DETAIL_PATH_BASE = 'http://lgtm.in/p/'.freeze
-LGTM_MARKDOWN_IMAGE_PATH_BASE = 'http://lgtm.in/i/'.freeze
-
-last_executed_at = begin File.read(BASE_PATH + '/' + LAST_EXECUTED_AT_TIME_PATH); rescue; nil;
-                   end
-
-cached_links = begin File.read(BASE_PATH + '/' + CACHED_LINKS_PATH); rescue; nil
-               end
-
-def fetch_links
-  agent = Mechanize.new
-  page = agent.get(LGTM_HAITO_PAGE)
-  page.links.map(&:href).select {|link| link.match(LGTM_LINK_BASE) }
-end
-
-def update_executed_at!
-  File.open(BASE_PATH + '/' + LAST_EXECUTED_AT_TIME_PATH, 'w') {|f| f.write(Time.now.to_i) }
-end
-
-def cache_links!(links)
-  File.open(BASE_PATH + '/' + CACHED_LINKS_PATH, 'w') {|f| f.write(Marshal.dump(links)) }
-end
-
-links =
-  if Time.now.to_i - last_executed_at.to_i > REFLASH_TIME
-    update_executed_at!
-
-    links = fetch_links
-    cache_links!(links)
-
-    links
-  else
-    cached_links.nil? ? fetch_links : Marshal.load(cached_links)
-  end
-
-link = links.sample
-
-hash = link.split('/').last
-
-lgtm_markdown = LGTM_MARKDOWN.gsub('__LGTM_IMAGE_DETAILS_PATH__', "#{LGTM_MARKDOWN_DETAIL_PATH_BASE + hash}").gsub('__LGTM_IMAGE_PATH__', "#{LGTM_MARKDOWN_IMAGE_PATH_BASE + hash}")
-
-system("echo '#{lgtm_markdown}' | pbcopy")
-