Allowing creation of timesheet records with begin and end time

[vworldat]

I added --begin/-b and --end/-e options to the kimai start command. This would resolve both #7 as well as parts of #8 (no --end option for kimai stop yet).

Had to add a phpstan baseline to ignore errors caused by kimai/api-php declaring nullable getters as non-nullable.

[kevinpapst]

I did not test it, but look good to me.

Thanks @vworldat 👍

[Copilot]

Pull request overview

Adds support for creating timesheet records with explicit begin/end timestamps via the kimai start command, enabling scripted/bulk inserts and “forgot to start/stop” workflows discussed in #7 and parts of #8.

Changes:

• Added --begin/-b and --end/-e options to kimai start, with interactive refinement for invalid date inputs.
• Updated CLI output to include an optional End timestamp.
• Added PHPStan baseline configuration to suppress a nullsafe-call finding originating from upstream API typings.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/Command/TimesheetCommandTrait.php	Adds a helper for parsing DateTime values (currently implemented recursively).
src/Command/StartCommand.php	Introduces --begin/--end options, parses them into the form, and prints End in output.
README.md	Documents new --begin/--end options with examples and refinement prompt behavior.
phpstan.neon	Includes the baseline file in PHPStan configuration.
phpstan-baseline.neon	Suppresses a specific PHPStan nullsafe-call rule for StartCommand.php.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[parthitce]

Thanks for this PR and it works. one fix is needed to make it to use the timezone setup of the machine instead of always UTC.

commit 9333efd642b9f72ff5a517eec3e48a4f608fcfe0 (HEAD -> build-2.0.0-pr26)
Author: Parthiban Nallathambi <parthiban@linumiz.com>
Date:   Sat Sep 19 20:00:49 2026 +0530

    Fix: interpret --begin/--end in the configured PHP timezone
    
    new DateTime($value) assumed the PHP process timezone implicitly,
    so backdated entries shifted by the UTC offset. Pass the value with
    date_default_timezone_get() so times match the configured zone.
    
    Signed-off-by: Parthiban Nallathambi <parthiban@linumiz.com>

diff --git a/src/Command/TimesheetCommandTrait.php b/src/Command/TimesheetCommandTrait.php
index 5ab15fc..fb315a8 100644
--- a/src/Command/TimesheetCommandTrait.php
+++ b/src/Command/TimesheetCommandTrait.php
@@ -328,7 +328,7 @@ trait TimesheetCommandTrait
     private function parseAndRefineDateTime(SymfonyStyle $io, string $begin, string $valueName): \DateTime
     {
         try {
-            return new \DateTime($begin);
+            return new \DateTime($begin, new \DateTimeZone(date_default_timezone_get()));
         } catch (\Exception $e) {
         }

It would be great if this PR can be merged.

[kevinpapst]

@parthitce do you want to take over and send an updated PR?

[parthitce]

@parthitce do you want to take over and send an updated PR?

Yeah sure. I will retain the original author and add the fix. may be additional signed-off. does that work?

[kevinpapst]

Did you check what happens if the current timezone is different from the API users timezone?
Shall we add an option to pass an explicit timezone?

[parthitce]

Did you check what happens if the current timezone is different from the API users timezone? Shall we add an option to pass an explicit timezone?

Addressed with #36 , IMO this PR can be closed