pit.nimble

@@ -1,6 +1,6 @@
 # Package
 
-version       = "4.29.0"
+version       = "4.28.0"
 author        = "Jonathan Bernard"
 description   = "Personal issue tracker."
 license       = "MIT"

src/pit.nim

@@ -67,9 +67,6 @@ when isMainModule:
   if args["--debug"]:
     consoleLogger.levelThreshold = lvlDebug
 
-  if args["--silent"]:
-    consoleLogger.levelThreshold = lvlNone
-
   if args["--echo-args"]: stderr.writeLine($args)
 
   if args["help"]:
@@ -81,7 +78,6 @@ when isMainModule:
 
   trace "context initiated"
 
-  var updatedIssues = newSeq[Issue]()
   var propertiesOption = none(TableRef[string,string])
   var exclPropsOption = none(TableRef[string,seq[string]])
   var tagsOption = none(seq[string])
@@ -137,7 +133,7 @@ when isMainModule:
         else: newSeq[string]())
 
     ctx.cfg.tasksDir.store(issue, state)
-    updatedIssues.add(issue)
+
     stdout.writeLine formatIssue(issue)
 
   elif args["reorder"]:
@@ -164,7 +160,6 @@ when isMainModule:
             for k,v in propsOption.get:
               issue[k] = v
           edit(issue)
-          updatedIssues.add(issue)
 
       else:
         let issue = ctx.cfg.tasksDir.loadIssueById(editRef)
@@ -172,7 +167,6 @@ when isMainModule:
           for k,v in propertiesOption.get:
             issue[k] = v
         edit(issue)
-        updatedIssues.add(issue)
 
   elif args["tag"]:
     if tagsOption.isNone: raise newException(Exception, "no tags given")
@@ -183,7 +177,6 @@ when isMainModule:
       var issue = ctx.cfg.tasksDir.loadIssueById(id)
       issue.tags = deduplicate(issue.tags & newTags)
       issue.store()
-      updatedIssues.add(issue)
 
   elif args["untag"]:
     let tagsToRemove: seq[string] =
@@ -197,7 +190,6 @@ when isMainModule:
           proc (tag: string): bool = not tagsToRemove.anyIt(it == tag))
       else: issue.tags = @[]
       issue.store()
-      updatedIssues.add(issue)
 
   elif args["start"] or args["todo-today"] or args["done"] or
        args["pending"] or args["todo"] or args["suspend"]:
@@ -221,12 +213,10 @@ when isMainModule:
           let nextIssue = ctx.cfg.tasksDir.nextRecurrence(issue.getRecurrence.get, issue)
           ctx.cfg.tasksDir.store(nextIssue, TodoToday)
           info "created the next recurrence:"
-          updatedIssues.add(nextIssue)
           stdout.writeLine formatIssue(nextIssue)
 
 
       issue.changeState(ctx.cfg.tasksDir, targetState)
-      updatedIssues.add(issue)
 
     if ctx.triggerPtk or args["--ptk"]:
       if targetState == Current:
@@ -251,7 +241,6 @@ when isMainModule:
     issue.setDateTime("hide-until", parseDate($args["<date>"]))
 
     issue.store()
-    updatedIssues.add(issue)
 
   elif args["delegate"]:
 
@@ -259,7 +248,6 @@ when isMainModule:
     issue["delegated-to"] = $args["<delegated-to>"]
 
     issue.store()
-    updatedIssues.add(issue)
 
   elif args["delete"] or args["rm"]:
     for id in @(args["<id>"]):
@@ -272,7 +260,6 @@ when isMainModule:
           continue
 
       issue.delete
-      updatedIssues.add(issue)
 
   elif args["list"]:
 
@@ -395,7 +382,6 @@ when isMainModule:
     finally: close(propIn)
 
     issue.store()
-    updatedIssues.add(issue)
 
   elif args["get-binary-property"]:
     let issue = ctx.cfg.tasksDir.loadIssueById($(args["<id>"]))
@@ -436,15 +422,6 @@ when isMainModule:
 
       sync(syncCtx, args["--dry-run"])
 
-  # after doing stuff, sync if auto-sync is requested
-  if ctx.cfg.autoSync:
-    for syncTarget in ctx.cfg.syncTargets:
-      let syncCtx = initSyncContext(ctx.cfg, syncTarget)
-      if anyIt(
-          updatedIssues,
-          it.hasProp("context") and it["context"] == syncCtx.issueContext):
-        sync(syncCtx, false)
-
  except CatchableError:
   fatal getCurrentExceptionMsg()
   debug getCurrentException().getStackTrace()

src/pit/cliconstants.nim

@@ -1,4 +1,4 @@
-const PIT_VERSION* = "4.29.0"
+const PIT_VERSION* = "4.28.0"
 
 const USAGE* = """Usage:
   pit ( new | add) <summary> [<state>] [options]
@@ -82,8 +82,6 @@ Options:
   --dry-run                 Currently only supported by the `sync` command:
                             only print the changes that would be made, but do
                             not actually make them.
-
-  -s, --silent              Suppress all logging and status output.
 """
 
 const ONLINE_HELP* = """Issue States:
@@ -191,4 +189,4 @@ Issue Properties:
 
       If present, expected to be a comma-delimited list of text tags. The -g
       option is a short-hand for '-p tags:<tags-value>'.
-"""
+"""

src/pit/formatting.nim

@@ -3,8 +3,6 @@ import cliutils, uuids
 import std/strutils except alignLeft, capitalize, strip, toLower, toUpper
 import ./libpit
 
-proc adjustedTerminalWidth(): int = min(terminalWidth(), 80)
-
 proc getIssueContextDisplayName*(ctx: CliContext, context: string): string =
   if not ctx.contexts.hasKey(context):
     if context.isEmptyOrWhitespace: return "<default>"
@@ -130,11 +128,11 @@ proc formatSectionIssueList*(
 
 proc formatSection(ctx: CliContext, issues: seq[Issue], state: IssueState,
                    indent = "", verbose = false): string =
-  let innerWidth = adjustedTerminalWidth() - (indent.len * 2)
+  let innerWidth = terminalWidth() - (indent.len * 2)
 
   result = termColor(fgBlue) &
     (indent & ".".repeat(innerWidth)) & "\n" &
-    state.displayName.center(adjustedTerminalWidth()) & "\n\n" &
+    state.displayName.center(terminalWidth()) & "\n\n" &
     termReset
 
   let issuesByContext = issues.groupBy("context")
@@ -154,9 +152,9 @@ proc formatSection(ctx: CliContext, issues: seq[Issue], state: IssueState,
 
 proc writeHeader*(ctx: CliContext, header: string) =
   stdout.setForegroundColor(fgRed, true)
-  stdout.writeLine('_'.repeat(adjustedTerminalWidth()))
-  stdout.writeLine(header.center(adjustedTerminalWidth()))
-  stdout.writeLine('~'.repeat(adjustedTerminalWidth()))
+  stdout.writeLine('_'.repeat(terminalWidth()))
+  stdout.writeLine(header.center(terminalWidth()))
+  stdout.writeLine('~'.repeat(terminalWidth()))
   stdout.resetAttributes
 
 

src/pit/libpit.nim

@@ -33,7 +33,6 @@ type
   PitConfig* = ref object
     tasksDir*: string
     contexts*: TableRef[string, string]
-    autoSync*: bool
     syncTargets*: seq[JsonNode]
     cfg*: CombinedConfig
 
@@ -460,7 +459,6 @@ proc loadConfig*(args: Table[string, Value] = initTable[string, Value]()): PitCo
 
   result = PitConfig(
     cfg: cfg,
-    autoSync: parseBool(cfg.getVal("auto-sync", "false")),
     contexts: newTable[string,string](),
     tasksDir: cfg.getVal("tasks-dir", ""),
     syncTargets: cfg.getJson("sync-targets", newJArray()).getElems)

src/pit/sync_pbm_vsb.nim

@@ -4,11 +4,11 @@ import timeutils, uuids, zero_functional
 import ./formatting, ./libpit
 
 type
-  PbmVsbSyncContext* = object
-    apiBaseUrl*: string
-    apiToken*: string
-    issueContext*: string
+  PbmVsbSyncContext = object
     pit: PitConfig
+    apiBaseUrl: string
+    apiToken: string
+    issueContext: string
     http: HttpClient
 
   ServerTask* = object
@@ -117,6 +117,7 @@ proc sync*(
     dryRun = true,
     batchSize = 100): void =
 
+  if not dryRun: echo "NOT DRY RUN"
   # We're going to do a uni-directional sync, pushing local issues to the
   # server. However, we only want to update issues that have changed since
   # the last sync based on the *last-updated* property.