timet 1.5.5 → 1.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc6553610acca8f30eb067accd9a8ff09756ea7c0fdb231dac896210018e0db1
-  data.tar.gz: 41e673fa16e1a7ea7d7c7c3f75a7131c7add0fa4ea297ae3044ca9ddbbd1b4f3
+  metadata.gz: 488c0136f2b11a62046a2f507b054d346a5def75dc3a32609ea51872bb411e6b
+  data.tar.gz: c28bbe32a4558ab6755e438ba7c5a53f879d6c036ae68706782b62d8ca26133c
 SHA512:
-  metadata.gz: '01793d4487cd53b78a2d35721a3e6d91ff29a8c98976e9e861bfc599e49dfbf89939bec66da4fd986f8b8f99c4cf2001ac859274859185aaf4a89f9f56bae7a3'
-  data.tar.gz: c0f27b5b4656c07a16e861544daf610b94d397a5b245e89d65850c23e24f1a5318042b6871430cd2c0341c0bdb3f27b4163781bbf024680157cd68889202459c
+  metadata.gz: d1e74dfa8e0d6fdacf0fd8c458ade7667e2f2722376aea9307a5dab7b68b6a0a561b475432cf666564ba6664c18b6f80f94aed0bd88c3f6703ead36aa5d1cf84
+  data.tar.gz: 70608773c6f504e38637b7d4b2d71a852d7ed20c5679170c082d302748e1e3e6e4be2d1d4b909ce49e2ae45052dc3ac2cd02ba9a51106aba723211d3efa3c66a

data/.rubocop.yml

@@ -6,7 +6,7 @@ AllCops:
     - timet.gemspec
 
 Metrics/MethodLength:
-  Max: 12
+  Max: 15
 
 plugins: rubocop-rspec
 

data/CHANGELOG.md

@@ -1,3 +1,60 @@
+## [1.5.7] - 2025-05-16
+
+**Improvements:**
+
+- Extracted collision checks in [`ValidationEditHelper`](lib/timet/validation_edit_helper.rb) into dedicated private methods (`check_collision_with_previous_item`, `check_collision_with_next_item`).
+- Enhanced time collision validation in [`ValidationEditHelper`](lib/timet/validation_edit_helper.rb): renamed `validate_collision` to `validate_time_collisions`, added a `field` argument for specific checks ('start'/'end'), implemented distinct collision logic, and updated `perform_validation` to use the new method for both fields.
+- Improved time validation logic structure by extracting `determine_start_base_date_time` and `determine_end_base_date_time` from `determine_base_date_time`, and introducing `validate_time_order` to centralize start/end time order validation.
+- Centralized datetime creation by introducing `create_datetime_from_components` in [`TimeHelper`](lib/timet/time_helper.rb).
+- Improved time validation and determination logic in [`TimeValidationHelper`](lib/timet/time_validation_helper.rb) and [`ValidationEditHelper`](lib/timet/validation_edit_helper.rb): refactored `determine_base_date_time` to use a `case` statement, extracted `validate_time_difference` for 24-hour difference validation, and removed the unused `time_str` parameter from `determine_and_create_datetime`.
+- Centralized time validation logic to the new [`TimeValidationHelper`](lib/timet/time_validation_helper.rb) module, moving relevant methods from [`ValidationEditHelper`](lib/timet/validation_edit_edit_helper.rb).
+- Refactored time validation and item data helpers by moving time validation logic to [`TimeUpdateHelper`](lib/timet/time_update_helper.rb) and item data fetching methods to [`ItemDataHelper`](lib/timet/item_data_helper.rb).
+- Extracted time parsing, base date determination, datetime creation, end time adjustment, and validation logic into dedicated private methods in [`lib/timet/validation_edit_helper.rb`](lib/timet/validation_edit_helper.rb).
+- Improved time validation and editing logic in [`ValidationEditHelper`](lib/timet/validation_edit_helper.rb): anchored end time date to start time date, handled end time on the next day, enforced < 24 hour duration, and preserved original time on empty input.
+- Centralized utility methods into a new [`Timet::Utils`](lib/timet/utils.rb) module.
+- Improved [`WeekInfo`](lib/timet/week_info.rb) initialization to accept a `Date` object and display string, and refactored its responsibilities for clarity and Reek compliance.
+- Improved [`TimeBlockChart`](lib/timet/time_block_chart.rb) readability by replacing magic numbers with constants and ensuring chronological date order in `print_blocks`.
+- Refactored [`TimeBlockChart`](lib/timet/time_block_chart.rb) for clarity and Reek compliance, moving separator printing responsibility and the `CHAR_MAPPING` constant to appropriate locations/helpers.
+- Refactored database initialization logic in [`Timet::Application`](lib/timet/application.rb) for clarity and testability, encapsulating `initialize_database` within a `no_commands` block.
+- Updated gem dependencies, including `aws-sdk-s3`, `icalendar`, `rubocop`, and `rubocop-rspec`.
+
+**Bug Fixes:**
+
+- Corrected `perform_validation` in [`ValidationEditHelper`](lib/timet/validation_edit_helper.rb) to accept and utilize the `item` object for correct context in collision checks.
+- Modified `create_new_datetime` to derive date components from the parsed time component, allowing users to specify a full date and time for 'start' or 'end' fields.
+- Fixed a bug in [`WeekInfo#format_and_print_date_info`](lib/timet/week_info.rb) where weekend days were incorrectly checked using "Sa" and "Su".
+- Handled empty `@time_block` in [`TimeBlockChart#initialize`](lib/timet/time_block_chart.rb) to prevent errors.
+
+## [1.5.6] - 2025-04-07
+
+**Improvements:**
+
+- **Enhanced Edit Command:** The `edit` command now interactively prompts the user for both the field to edit and its new value if they are not provided as command-line arguments.
+- **Improved Database Initialization:** Database initialization logic has been extracted into a separate `initialize_database` method within the `Application` class, improving code organization and readability.
+- **Refactored `validate_and_update`:** The `validate_and_update` method in `ValidationEditHelper` has been refactored to handle both time fields and other fields more robustly. It now raises an `ArgumentError` for invalid fields and returns the updated item.
+- **Enhanced Security:** The `update_item` method in the `Database` class now utilizes parameterized queries to prevent SQL injection vulnerabilities.
+- **Improved Data Integrity:** The `update_time_columns` method in the `Database` class now correctly updates the `updated_at` and `created_at` columns to reflect the current time.
+- **Code Clarity:** Increased the `Metrics/MethodLength` max length to 15 in `.rubocop.yml` to accommodate refactored code.
+- **Dependency Updates:**
+  - Updated `aws-sdk-s3` to version 1.183.
+  - Updated `csv` to version 3.3.3.
+  - Updated `diff-lcs` to version 1.6.1.
+  - Updated `json` to version 2.10.2.
+  - Updated `parser` to version 3.3.7.4.
+  - Updated `rubocop` to version 1.75.2.
+  - Updated `rubocop-ast` to version 1.44.0.
+  - Added `prism` to version 1.4.0.
+  - Added `logger` to version 1.7.0.
+- **Test Coverage:** Added new tests to `application_spec.rb` to cover the `edit` command, and to `database_spec.rb` to cover the `update_item` and `update_time_columns` methods. Refactored tests in `validation_edit_helper_spec.rb` to cover changes in `validate_and_update`.
+- **Git Ignore:** Added `.qodo` to `.gitignore` to prevent it from being added to the repository.
+
+**Bug Fixes:**
+
+- **Edit Command:** The `edit` command now correctly updates the database with the new value.
+- **`update_item`:** The `update_item` method is no longer vulnerable to SQL injection.
+- **`update_time_columns`:** The `update_time_columns` method now correctly updates the `updated_at` and `created_at` columns.
+- **.qodo file:** The .qodo file is now ignored by git.
+
 ## [1.5.5] - 2025-02-26
 
 **Improvements:**

data/lib/timet/application.rb

@@ -51,16 +51,43 @@ module Timet
 
     def initialize(*args)
       super
+      config = args[2] || {} # Third argument is the config hash
+      initialize_database(config)
+    end
+
+    # Initializes the database connection based on the provided options.
+    #
+    # This method determines how to initialize the database connection based on the options provided.
+    # It supports injecting a database instance for testing, using a fallback for RSpec, and initializing
+    # a production database based on valid command arguments.
+    #
+    # @param options [Hash] A hash of options that may include a :database key for injecting a database instance.
+    # @option options [Database] :database An instance of the Database class to be used directly.
+    # @option options [Hash] :current_command The current command being executed, used to validate production
+    #   database initialization.
+    #
+    # @return [void] This method does not return a value; it initializes the `@db` instance variable.
+    #
+    # @raise [SystemExit] If invalid arguments are provided for production database initialization.
+    no_commands do
+      def initialize_database(options)
+        db_from_options = options[:database]
 
-      if defined?(RSpec)
-        @db = Database.new
-      else
-        command_name = args.dig(2, :current_command, :name)
-        if VALID_ARGUMENTS.include?(command_name)
+        # Allow injecting a database instance, primarily for testing
+        if db_from_options
+          @db = db_from_options
+        elsif defined?(RSpec)
+          # Fallback for RSpec if not injected (though injection is preferred)
           @db = Database.new
         else
-          warn 'Invalid arguments provided. Please check your input.'
-          exit(1)
+          # Production database initialization
+          command_name = options.dig(:current_command, :name)
+          if VALID_ARGUMENTS.include?(command_name)
+            @db = Database.new
+          else
+            warn 'Invalid arguments provided. Please check your input.'
+            exit(1)
+          end
         end
       end
     end
@@ -219,13 +246,14 @@ module Timet
       return puts "No tracked time found for id: #{id}" unless item
 
       display_item(item)
-      unless FIELD_INDEX.keys.include?(field&.downcase) || new_value
+      if field.nil? || new_value.nil?
         field = select_field_to_edit
         new_value = prompt_for_new_value(item, field)
       end
 
       updated_item = validate_and_update(item, field, new_value)
-      display_item(updated_item || item)
+      @db.update_item(id, field, updated_item[FIELD_INDEX[field]])
+      display_item(updated_item)
     end
 
     desc 'delete (d) [id]', 'Delete task => tt d 23'

data/lib/timet/block_char_helper.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module Timet
+  #
+  # The BlockCharHelper module provides a utility method for getting the block character
+  # based on a given value.
+  #
+  module BlockCharHelper
+    # Character mapping for different time ranges
+    CHAR_MAPPING = {
+      0..120 => '_',
+      121..450 => ' ',
+      451..900 => '▂',
+      901..1350 => '▃',
+      1351..1800 => '▄',
+      1801..2250 => '▅',
+      2251..2700 => '▆',
+      2701..3150 => '▇',
+      3151..3600 => '█'
+    }.freeze
+
+    # Gets the block character for a given value
+    #
+    # @param [Integer, nil] value The value
+    # @return [String] The block character
+    def self.get_block_char(value)
+      return ' ' unless value
+
+      mapping = CHAR_MAPPING.find { |range, _| range.include?(value) }
+      mapping ? mapping.last : ' '
+    end
+  end
+end

data/lib/timet/database.rb

@@ -115,9 +115,7 @@ module Timet
     #
     # @note The method executes SQL to update the specified field of the item with the given ID.
     def update_item(id, field, value)
-      return if %w[start end].include?(field) && value.nil?
-
-      execute_sql("UPDATE items SET #{field}='#{value}', updated_at=#{Time.now.utc.to_i} WHERE id = #{id}")
+      execute_sql("UPDATE items SET #{field} = ?, updated_at = ? WHERE id = ?", [value, Time.now.utc.to_i, id])
     end
 
     # Deletes an item from the items table.
@@ -162,19 +160,21 @@ module Timet
       result.empty? ? nil : result[0]
     end
 
-    # Finds an item in the items table by its ID.
+    # Finds an item by its ID.
     #
-    # @param id [Integer] The ID of the item to be found.
+    # @param id [Integer] The ID of the item to find.
     #
-    # @return [Array, nil] The item as an array, or nil if the item does not exist.
+    # @return [Array, nil] The item as an array if found, nil otherwise.
     #
     # @example Find an item with ID 1
-    #   find_item(1)
+    #   find_item(1) # => [1, 1678886400, 1678890000, 'work', 'notes', nil, 1678890000, 1678886400, nil]
     #
-    # @note The method executes SQL to find the item with the given ID in the 'items' table.
+    # @note The method executes a SQL query to find the item by its ID.
+    # @note If the item is found, it returns the item as an array.
+    # @note If the item is not found, it returns nil.
     def find_item(id)
-      result = execute_sql('SELECT * FROM items WHERE id = ? AND (deleted IS NULL OR deleted = 0)', [id])
-      result.empty? ? nil : result[0]
+      result = execute_sql('SELECT * FROM items WHERE id = ?', [id])
+      result.first.dup if result.any? # Add .dup to create a copy
     end
 
     # Fetches all items from the items table that have a start time greater than or equal to today.
@@ -316,7 +316,7 @@ module Timet
       result.each do |item|
         id = item[0]
         end_time = item[2]
-        execute_sql("UPDATE items SET updated_at = #{end_time}, created_at = #{end_time} WHERE id = #{id}")
+        execute_sql('UPDATE items SET updated_at = ?, created_at = ? WHERE id = ?', [end_time, end_time, id])
       end
     end
   end

data/lib/timet/item_data_helper.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module Timet
+  # Helper methods for fetching item data.
+  module ItemDataHelper
+    module_function
+
+    # Fetches the start time of a tracking item.
+    #
+    # @param item [Array] The tracking item.
+    #
+    # @return [Integer] The start time in epoch format.
+    #
+    # @example Fetch the start time of a tracking item
+    #   fetch_item_start(item)
+    def fetch_item_start(item)
+      item[Timet::Application::FIELD_INDEX['start']]
+    end
+
+    # Fetches the end time of a tracking item.
+    #
+    # @param item [Array] The tracking item.
+    #
+    # @return [Integer] The end time in epoch format.
+    #
+    # @example Fetch the end time of a tracking item
+    #   fetch_item_end(item)
+    def fetch_item_end(item)
+      item[Timet::Application::FIELD_INDEX['end']] || TimeHelper.current_timestamp
+    end
+
+    # Fetches the end time of the tracking item before the current one.
+    #
+    # @param db [Timet::Database] The database instance.
+    # @param id [Integer] The ID of the current tracking item.
+    # @param item_start [Integer] The start time of the current tracking item.
+    #
+    # @return [Integer] The end time of the previous tracking item in epoch format.
+    #
+    # @example Fetch the end time of the previous tracking item
+    #   fetch_item_before_end(db, 1, 1633072800)
+    def fetch_item_before_end(db, id, item_start)
+      db.find_item(id - 1)&.dig(Timet::Application::FIELD_INDEX['end']) || item_start
+    end
+
+    # Fetches the start time of the tracking item after the current one.
+    #
+    # @param db [Timet::Database] The database instance.
+    # @param id [Integer] The ID of the current tracking item.
+    #
+    # @return [Integer] The start time of the next tracking item in epoch format.
+    #
+    # @example Fetch the start time of the next tracking item
+    #   fetch_item_after_start(db, id)
+    def fetch_item_after_start(db, id)
+      db.find_item(id + 1)&.dig(Timet::Application::FIELD_INDEX['start']) || TimeHelper.current_timestamp
+    end
+  end
+end

data/lib/timet/table.rb

@@ -2,6 +2,7 @@
 
 # require_relative 'color_codes'
 require 'timet/time_report_helper'
+require_relative 'utils'
 module Timet
   # This class is responsible for formatting the output of the `timet` application.
   # It provides methods for formatting the table header, separators, and rows.
@@ -123,7 +124,7 @@ module Timet
 
       block_hour = TimeHelper.count_seconds_per_hour_block(start_time, end_time, tag)
       date_line = TimeHelper.timestamp_to_date(start_time)
-      time_block[date_line] = add_hashes(time_block[date_line], block_hour)
+      time_block[date_line] = Timet::Utils.add_hashes(time_block[date_line], block_hour)
       time_block
     end
 

data/lib/timet/time_block_chart.rb

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require_relative 'week_info'
+require_relative 'block_char_helper'
+
 module Timet
   #
   # The TimeBlockChart class is responsible for generating and printing a visual representation
@@ -18,22 +21,18 @@ module Timet
   # @attr_reader [Integer] start_hour The starting hour of the time block
   # @attr_reader [Integer] end_hour The ending hour of the time block
   class TimeBlockChart
-    # Character mapping for different time ranges
-    CHAR_MAPPING = {
-      0..120 => '_',
-      121..450 => '▁',
-      451..900 => '▂',
-      901..1350 => '▃',
-      1351..1800 => '▄',
-      1801..2250 => '▅',
-      2251..2700 => '▆',
-      2701..3150 => '▇',
-      3151..3600 => '█'
-    }.freeze
-
     # Separator character for the chart
     SEPARATOR_CHAR = '░'
 
+    # Width of the date/week string content (e.g., "02 2023-10-01 Fri")
+    DATE_WEEK_CONTENT_WIDTH = 17
+
+    # Width of the '┆- ' part after the date/week string
+    DATE_WEEK_BORDER_WIDTH = 3
+
+    # Width of the total hours column including the border
+    TOTAL_HOURS_COLUMN_WIDTH = 4
+
     # Initializes a new TimeBlockChart instance.
     #
     # This method sets up the time block chart by processing the time entries from the provided table
@@ -50,8 +49,13 @@ module Timet
     # @see Table#process_time_entries
     def initialize(table)
       @time_block = table.process_time_entries(display: false)
-      @start_hour = @time_block.values.map(&:keys).flatten.uniq.min.to_i
-      @end_hour = @time_block.values.map(&:keys).flatten.uniq.max.to_i
+      hours = @time_block.values.map(&:keys).flatten.uniq
+      if hours.empty?
+        @start_hour = @end_hour = 0
+        return
+      end
+      @start_hour = hours.min.to_i
+      @end_hour   = hours.max.to_i
     end
 
     # Prints the time block chart.
@@ -59,13 +63,14 @@ module Timet
     # This method formats and prints the time block chart, including the header and the time blocks
     # for each entry. The chart is color-coded based on the provided color mapping for different tags.
     #
-    # @param table [Hash] The time block data to be displayed in the chart.
     # @param colors [Hash] A mapping of tags to colors, used to color-code the time blocks.
     # @return [void] This method does not return a value; it performs side effects such as printing the chart.
     #
     # @example Print a time block chart
+    #   # Assuming 'table' is an instance of Timet::Table
     #   chart = TimeBlockChart.new(table)
-    #   chart.print_time_block_chart(table, colors)
+    #   colors = { "work" => 0, "break" => 1 } # Example color mapping
+    #   chart.print_time_block_chart(colors)
     #
     # @note
     #   - The method first prints the header of the chart, which includes the time range.
@@ -74,9 +79,11 @@ module Timet
     #
     # @see #print_header
     # @see #print_blocks
-    def print_time_block_chart(table, colors)
+    def print_time_block_chart(colors)
+      return puts 'No time-block data to display.' if @no_data
+
       print_header
-      print_blocks(table, colors)
+      print_blocks(colors)
     end
 
     private
@@ -86,48 +93,64 @@ module Timet
     # @return [void]
     def print_header
       puts
-      print ' ' * 19
-      (@start_hour..@end_hour + 1).each { |hour| print format('%02d', hour).rjust(4) }
-      puts
-      puts '┌╴W ╴╴╴╴╴╴⏰╴╴╴╴╴╴┼'.gray + "#{'╴' * (@end_hour - @start_hour + 1) * 4}╴╴╴┼".gray
+      print_hours_row
+      # Dynamically build header using width constants
+      left = "┌#{'╴' * (DATE_WEEK_CONTENT_WIDTH - 11)}"
+      week = 'W'.center(DATE_WEEK_BORDER_WIDTH, '╴')
+      clock = '⏰'.center(DATE_WEEK_BORDER_WIDTH + 4, '╴')
+      hours_width = (@end_hour - @start_hour + 1) * 4
+      hours = '╴' * hours_width
+      right = "#{'╴' * (TOTAL_HOURS_COLUMN_WIDTH - 1)}┼"
+      header = left.gray + week.gray + clock.gray + '┼'.gray + hours.gray + right.gray
+      puts header
     end
 
-    # Prints the time blocks for each date in the time block data.
+    # Prints the hours row in the header.
     #
-    # This method iterates over the time block data, formats and prints the date information,
-    # prints the time blocks for each date using the provided color mapping, and calculates
-    # and prints the total hours for each day. It also prints a footer at the end.
+    # @return [void]
+    def print_hours_row
+      left_margin = DATE_WEEK_CONTENT_WIDTH + DATE_WEEK_BORDER_WIDTH - 1
+      print ' ' * left_margin
+      (@start_hour..@end_hour).each { |hour| print format('%02d', hour).rjust(4) }
+      puts
+    end
+
+    # Prints the main body of the time block chart, including date rows and corresponding time blocks.
     #
-    # @param table [Hash] The time block data containing the time entries for each date.
-    # @param colors [Hash] A mapping of tags to colors, used to color-code the time blocks.
-    # @return [void] This method does not return a value; it performs side effects such as printing
-    #   the time blocks and related information.
+    # This method iterates over each date present in the `@time_block` instance variable,
+    # which is expected to be a hash mapping date strings to hourly time block data.
+    # For each date, it:
+    #   1. Displays week and date information using `WeekInfo`.
+    #   2. Prints the visual time blocks for the hours of the day using `#print_time_blocks`.
+    #   3. Calculates and prints the total hours logged for that day using `#calculate_and_print_hours`.
+    # After processing all dates, it prints a chart footer using `#print_footer`.
     #
-    # @example Print time blocks
-    #   chart = TimeBlockChart.new(table)
-    #   chart.print_blocks(table, colors)
+    # @param colors [Hash] A mapping of tags to color indices. This is used by `#print_time_blocks`
+    #   to color-code the visual time blocks. Example: `{ "work" => 0, "break" => 1 }`
+    # @return [void] This method does not return a value; it prints directly to the console.
     #
     # @note
-    #   - The method skips processing if the `table` parameter is `nil`.
-    #   - For each date in the time block data, it formats and prints the date and day of the week.
-    #   - It prints the time blocks using the provided color mapping to visually distinguish
-    #     between different tags.
-    #   - It calculates and prints the total hours for each day.
-    #   - A footer is printed at the end to provide a visual separation.
+    #   - This is a private method, primarily called by `#print_time_block_chart`.
+    #   - It returns early if `@time_block` is `nil`. If `@time_block` is an empty hash,
+    #     no date-specific rows are printed, but the chart footer is still rendered.
+    #   - Relies on `@start_hour` and `@end_hour` instance variables being correctly set
+    #     during the `TimeBlockChart`'s initialization.
     #
-    # @see #format_and_print_date_info
+    # @see WeekInfo#format_and_print_date_info
     # @see #print_time_blocks
     # @see #calculate_and_print_hours
     # @see #print_footer
-    def print_blocks(table, colors)
-      return unless table
+    def print_blocks(colors)
+      return unless @time_block
 
       weeks = []
-      @time_block.each_key do |date_string|
-        date = Date.parse(date_string)
-        day = date.strftime('%a')[0..2]
+      @time_block.keys.sort.each do |date_string| # ISO-date strings sort naturally
+        date_object = Date.parse(date_string)
+        day = date_object.strftime('%a')[0..2]
 
-        format_and_print_date_info(date_string, day, weeks)
+        week_info = WeekInfo.new(date_object, date_string, weeks)
+        print_inter_week_separator if week_info.needs_inter_week_separator?
+        week_info.format_and_print_date_info(day)
 
         time_block_initial = @time_block[date_string]
         print_time_blocks(time_block_initial, colors)
@@ -148,62 +171,6 @@ module Timet
       puts
     end
 
-    # Formats and prints the date information
-    #
-    # @param [String] date_string The date string
-    # @param [String] day The day of the week
-    # @param [Array] weeks The list of weeks
-    # @return [void]
-    def format_and_print_date_info(date_string, day, weeks)
-      weekend = date_string
-      day = day.red if %w[Sa Su].include?(day)
-      weekend = weekend.red if %w[Sa Su].include?(day)
-
-      week = format_and_print_week(date_string, weeks)
-
-      print '┆'.gray + "#{week} #{weekend} #{day}" + '┆- '.gray
-    end
-
-    # Formats and prints the week information
-    #
-    # @param [String] date_string The date string
-    # @param [Array] weeks The list of weeks
-    # @return [String] The formatted week string
-    def format_and_print_week(date_string, weeks)
-      week, current_index = determine_week(date_string, weeks)
-      print_separator(week, current_index)
-      week
-    end
-
-    # Determines the week for a given date
-    #
-    # @param [String] date_string The date string
-    # @param [Array] weeks The list of weeks
-    # @return [Array] The week string and current index
-    def determine_week(date_string, weeks)
-      weeks << Date.parse(date_string).cweek
-      current_index = weeks.size - 1
-      current_week = weeks[current_index]
-      week = if current_week == weeks[current_index - 1] && current_index.positive?
-               '  '
-             else
-               format('%02d', current_week).to_s.underline
-             end
-      [week, current_index]
-    end
-
-    # Prints the separator line
-    #
-    # @param [String] week The week string
-    # @param [Integer] current_index The current index
-    # @return [void]
-    def print_separator(week, current_index)
-      return unless week != '  ' && current_index.positive?
-
-      sep = SEPARATOR_CHAR
-      puts "┆#{sep * 17}┼#{sep * (@end_hour - @start_hour + 1) * 4}#{sep * 3}┼#{sep * 4}".gray
-    end
-
     # Prints the footer of the chart
     #
     # @return [void]
@@ -234,7 +201,7 @@ module Timet
       formatted_hour = format('%02d', hour)
       hour_data = time_block_initial[formatted_hour]
       tag = hour_data&.last
-      [tag, get_block_char(hour_data&.first)]
+      [tag, BlockCharHelper.get_block_char(hour_data&.first)]
     end
 
     # Prints the colored block character
@@ -250,14 +217,21 @@ module Timet
       print colored_block.rjust(4)
     end
 
-    # Gets the block character for a given value
+    # Prints the separator line between different weeks in the chart.
     #
-    # @param [Integer, nil] value The value
-    # @return [String] The block character
-    def get_block_char(value)
-      return ' ' unless value
-
-      CHAR_MAPPING.find { |range, _| range.include?(value) }&.last || ' '
+    # @return [void]
+    def print_inter_week_separator
+      sep = SEPARATOR_CHAR
+      line_parts = [
+        '┆',
+        sep * DATE_WEEK_CONTENT_WIDTH,                # Date/Week column fill
+        '┼',
+        sep * (@end_hour - @start_hour + 1) * 4,      # Hour blocks column fill
+        sep * DATE_WEEK_BORDER_WIDTH,                 # Additional fill for hour blocks column
+        '┼',
+        sep * TOTAL_HOURS_COLUMN_WIDTH                # Total hours column fill
+      ]
+      puts line_parts.join.gray
     end
   end
 end

data/lib/timet/time_helper.rb

@@ -247,5 +247,22 @@ module Timet
     def self.beginning_of_day(time = Time.now)
       Time.new(time.year, time.month, time.day)
     end
+
+    # Creates a new datetime object.
+    #
+    # @param base_date_time [Time] The base date and time.
+    # @param parsed_time_component [Time] The parsed time component.
+    #
+    # @return [Time] The new datetime object.
+    def self.create_new_datetime(base_date_time, parsed_time_component)
+      Time.new(
+        base_date_time.year,
+        base_date_time.month,
+        base_date_time.day,
+        parsed_time_component.hour,
+        parsed_time_component.min,
+        parsed_time_component.sec
+      )
+    end
   end
 end