harvest-worklog 0.9.0 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ead4ee3da24a3522c5b5e09e694de573b4e10c2736b7e328377197efd349dcb
-  data.tar.gz: 41cce9ae00a04e151fd68914ddbb3200ae8451c837f93e95a8bd5b601ea1fc27
+  metadata.gz: 59f6ddf4acd09bb36f6f508719fccdd81db151107203367cd85deec4fb4bb6da
+  data.tar.gz: ebabb8af8524ca17fa2e1972582853eaaef6c721a62df4ad68671091cd63a65f
 SHA512:
-  metadata.gz: 2e2592d645f2153fab30539d0ad0fd103b8008c0025950c9222298fc1e06abf4cbd759f3b1a6e4a8eeba5ecf985670d13b994d430b870144afdf825a0ea44018
-  data.tar.gz: 11f9e301054fc944a9abf5a4509777cb9e6e538d3cd5be2fd50fc06b2a88feaf577b7ae22265ffe7d0a45972f5a7d25f58b99fb9ea36fd54b9eaee2edce0cd64
+  metadata.gz: 637f2eebaad54a58e481b9b860f6af18af769fb573c0af55843e7197a6e02101b4b8aeea998f99a0a2b81bde1d4606c0725715bcf676d51fabe9a025b58fb688
+  data.tar.gz: 2ce2096247b4b779a6e88615748d9df27c67841ffaf2dc6f8d70874deaa8ceafc8e7f4ff84f196bb970863932f2ad8eb5c97b314c5564667a068ae93119f0a1c

data/README.md

@@ -21,7 +21,9 @@ Personal Access Tokens are the intended Harvest authentication method for person
 
 ```text
 harvest-worklog time-off FROM TO --project NAME --task NAME [options]
+harvest-worklog time-off FROM TO --project-id ID --task-id ID [options]
 harvest-worklog work-entry DATE --project NAME --task NAME --hours HOURS --notes NOTES [options]
+harvest-worklog work-entry DATE --project-id ID --task-id ID --hours HOURS --notes NOTES [options]
 harvest-worklog aggregate FROM TO [--project NAME] [--task NAME]
 harvest-worklog timesheet DATE --project NAME [--task NAME]
 ```
@@ -42,6 +44,8 @@ The default holiday region is `ca_yt`. Add `--holiday-region REGION` or set `HAR
 
 A time-off block never creates weekend entries. When names are supplied, it resolves them from your active personal Harvest assignments, so a Project Manager role is not required.
 
+The `harvest_record_time_off` OMP tool accepts the same name pair or ID pair. Its optional `holidayRegions` array adds per-call regions to the configured defaults.
+
 ### Reviewed ordinary work
 
 The `work-entry` command checks for existing or locked entries for its project, task, and date before creating a new duration entry. Use `--dry-run` to complete that preflight without a write. OMP Project Time uses the same path.
@@ -60,15 +64,16 @@ The `harvest_time_aggregates` OMP tool exposes the same optional filters and is
 
 ### Daily timesheets
 
-The `timesheet` command reads the authenticated user's compact daily Harvest view: task totals, entry durations when a task has multiple entries, and multiline notes. `DATE` accepts `today`, `yesterday`, or an ISO date in the CLI's local timezone.
+The CLI `timesheet` command and `harvest_time_sheet` OMP tool read the authenticated user's compact daily Harvest view. They show task totals, entry durations, and notes already recorded in Harvest.
 
 ```bash
 harvest-worklog timesheet today --project WRAP
 ```
 
-OMP exposes the same CLI-only path through `/harvest-worklog today WRAP` and the read-only `harvest_time_sheet` tool. The wrapper never calls Harvest directly. A later, separate feature may upload reviewed content through the Harvest API; keep that write path separate from this read-only formatter.
+The `/harvest-worklog timesheet today --project wrap` slash command instead reads local OMP Project Time and reports separate `Human active` and `Agent elapsed` totals. It does not call Harvest or require `projectTimeMappings`; mappings remain specific to previewing or recording Harvest entries.
+`/project-time history` reports all logged dates, so its totals can be larger than a single-day timesheet.
 
-Type `/harvest-worklog ` in OMP to discover the compact and explicit timesheet forms, relative date tokens, and context-valid `--project`/`--task` options. Completion keeps filter values as editable text, and the slash command delegates the selected timesheet argument vector to the CLI.
+Type `/harvest-worklog ` in OMP to discover `timesheet`, then select `today`, `yesterday`, or an ISO date before the contextual `--project` option. The editable project name must exactly match the case-sensitive local Project Time name.
 
 ## OMP settings
 

data/lib/harvest_worklog/aggregate_cli.rb

@@ -8,7 +8,7 @@ module HarvestWorklog
       options = {}
       parser = option_parser(options)
       dates = parser.parse(arguments)
-      validate!(dates)
+      validate!(dates, options)
 
       from = Date.iso8601(dates[0])
       to = Date.iso8601(dates[1])
@@ -36,8 +36,10 @@ module HarvestWorklog
       end
     end
 
-    def self.validate!(dates)
+    def self.validate!(dates, options = {})
       raise Error, "FROM and TO are required" unless dates.length == 2
+      raise Error, "--project must not be blank" if options[:project]&.strip&.empty?
+      raise Error, "--task must not be blank" if options[:task]&.strip&.empty?
     end
 
     def self.fetch_entries(client, from:, to:, user_id: nil)

data/lib/harvest_worklog/timesheet_cli.rb

@@ -46,7 +46,8 @@ module HarvestWorklog
 
     def self.validate!(dates, options)
       raise Error, "DATE is required" unless dates.length == 1
-      raise Error, "--project is required" unless options[:project] && !options[:project].empty?
+      raise Error, "--project is required" unless options[:project] && !options[:project].strip.empty?
+      raise Error, "--task must not be blank" if options[:task]&.strip&.empty?
     end
 
     def self.print_timesheet(output, entries, spent_date:, requested_project:)

data/lib/harvest_worklog/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HarvestWorklog
-  VERSION = "0.9.0"
+  VERSION = "0.10.1"
 end

data/lib/harvest_worklog/work_entry_cli.rb

@@ -42,7 +42,9 @@ module HarvestWorklog
       end
 
       if options[:dry_run]
-        output.puts "Would create #{spent_date.iso8601}: #{HarvestWorklog.display_hours(options[:hours])}h on #{options[:project]} / #{options[:task]}; #{options[:notes]}"
+        project = options[:project] || "project ##{options[:project_id]}"
+        task = options[:task] || "task ##{options[:task_id]}"
+        output.puts "Would create #{spent_date.iso8601}: #{HarvestWorklog.display_hours(options[:hours])}h on #{project} / #{task}; #{options[:notes]}"
         return 0
       end
 
@@ -63,7 +65,10 @@ module HarvestWorklog
 
     def self.option_parser(options)
       OptionParser.new do |opts|
-        opts.banner = "Usage: harvest-worklog work-entry DATE --project NAME --task NAME --hours HOURS --notes NOTES [--dry-run]"
+        opts.banner = <<~USAGE
+          Usage: harvest-worklog work-entry DATE --project NAME --task NAME --hours HOURS --notes NOTES [options]
+                 harvest-worklog work-entry DATE --project-id ID --task-id ID --hours HOURS --notes NOTES [options]
+        USAGE
         opts.on("--project NAME", "Harvest project name") { |value| options[:project] = value }
         opts.on("--task NAME", "Harvest task name") { |value| options[:task] = value }
         opts.on("--project-id ID", Integer, "Harvest project ID") { |value| options[:project_id] = value }
@@ -82,12 +87,12 @@ module HarvestWorklog
     def self.validate!(dates, options)
       raise Error, "DATE is required" unless dates.length == 1
       raise Error, "--hours must be a positive finite number" unless options[:hours]&.positive? && options[:hours].finite?
-      raise Error, "--notes is required" unless options[:notes] && !options[:notes].empty?
+      raise Error, "--notes is required" unless options[:notes] && !options[:notes].strip.empty?
 
       names_provided = options[:project] || options[:task]
       ids_provided = options[:project_id] || options[:task_id]
-      valid_names = options[:project] && options[:task]
-      valid_ids = options[:project_id] && options[:task_id]
+      valid_names = options[:project] && !options[:project].strip.empty? && options[:task] && !options[:task].strip.empty?
+      valid_ids = options[:project_id]&.positive? && options[:task_id]&.positive?
       raise Error, "supply --project and --task, or --project-id and --task-id" unless valid_names || valid_ids
       raise Error, "do not combine project/task names with IDs" if names_provided && ids_provided
     end

data/lib/harvest_worklog.rb

@@ -52,7 +52,9 @@ module HarvestWorklog
       <<~USAGE
         Usage:
           harvest-worklog time-off FROM TO --project NAME --task NAME [options]
+          harvest-worklog time-off FROM TO --project-id ID --task-id ID [options]
           harvest-worklog work-entry DATE --project NAME --task NAME --hours HOURS --notes NOTES [options]
+          harvest-worklog work-entry DATE --project-id ID --task-id ID --hours HOURS --notes NOTES [options]
           harvest-worklog aggregate FROM TO [--project NAME] [--task NAME]
           harvest-worklog timesheet DATE --project NAME [--task NAME]
 
@@ -70,6 +72,7 @@ module HarvestWorklog
       options = { hours: 7.0, dry_run: false, holiday_regions: holiday_regions_from_environment }
       parser = option_parser(options)
       dates = parser.parse(arguments)
+      normalize_holiday_regions!(options[:holiday_regions])
       validate!(dates, options)
 
       from = Date.iso8601(dates[0])
@@ -133,11 +136,12 @@ module HarvestWorklog
       raise Error, "FROM and TO are required" unless dates.length == 2
       raise Error, "--hours must be a positive finite number" unless options[:hours].positive? && options[:hours].finite?
       raise Error, "--holiday-region or HARVEST_HOLIDAY_REGIONS is required" if options[:holiday_regions].empty?
+      raise Error, "--notes must not be blank" if options[:notes]&.strip&.empty?
 
       names_provided = options[:project] || options[:task]
       ids_provided = options[:project_id] || options[:task_id]
-      valid_names = options[:project] && options[:task]
-      valid_ids = options[:project_id] && options[:task_id]
+      valid_names = options[:project] && !options[:project].strip.empty? && options[:task] && !options[:task].strip.empty?
+      valid_ids = options[:project_id]&.positive? && options[:task_id]&.positive?
       raise Error, "supply --project and --task, or --project-id and --task-id" unless valid_names || valid_ids
       raise Error, "do not combine project/task names with IDs" if names_provided && ids_provided
     end
@@ -146,6 +150,13 @@ module HarvestWorklog
       ENV.fetch("HARVEST_HOLIDAY_REGIONS", "ca_yt").split(",").map { |region| region.strip.downcase }.reject(&:empty?)
     end
 
+    def self.normalize_holiday_regions!(regions)
+      regions.map! { |region| region.strip.downcase }
+      regions.reject!(&:empty?)
+      regions.uniq!
+      regions
+    end
+
     def self.resolve_assignment(client, project_name, task_name)
       # ponytail: one 2,000-item page covers personal assignments; follow cursor pagination if that ceiling is exceeded.
       matches = client.active_personal_task_assignments.select do |assignment|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: harvest-worklog
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.1
 platform: ruby
 authors:
 - Marlen Brunner