Cap project delays

Author: Thomas Stromberg

.gitignore

@@ -32,3 +32,6 @@ go.work.sum
 # .vscode/
 
 .claude/
+
+# added by lint-install
+out/

.golangci.yml

@@ -0,0 +1,240 @@
+## Golden config for golangci-lint - strict, but within the realm of what Go authors might use.
+#
+# This is tied to the version of golangci-lint listed in the Makefile, usage with other
+# versions of golangci-lint will yield errors and/or false positives.
+#
+# Docs: https://github.com/golangci/golangci-lint/blob/HEAD/.golangci.reference.yml
+# Based heavily on https://gist.github.com/maratori/47a4d00457a92aa426dbd48a18776322
+
+version: "2"
+
+issues:
+  # Maximum issues count per one linter. Set to 0 to disable. Default is 50.
+  max-issues-per-linter: 0
+  # Maximum count of issues with the same text. Set to 0 to disable. Default is 3.
+  max-same-issues: 0
+
+formatters:
+  enable:
+    # - gci
+    # - gofmt
+    - gofumpt
+    # - goimports
+    # - golines
+    - swaggo
+
+  settings:
+    golines:
+      # Default: 100
+      max-len: 120
+
+linters:
+  default: all
+  disable:
+    # linters that give advice contrary to what the Go authors advise
+    - decorder # checks declaration order and count of types, constants, variables and functions
+    - dupword # [useless without config] checks for duplicate words in the source code
+    - exhaustruct # [highly recommend to enable] checks if all structure fields are initialized
+    - forcetypeassert # [replaced by errcheck] finds forced type assertions
+    - ginkgolinter # [if you use ginkgo/gomega] enforces standards of using ginkgo and gomega
+    - gochecknoglobals # checks that no global variables exist
+    - cyclop # replaced by revive
+    - gocyclo # replaced by revive
+    - forbidigo # needs configuration to be useful
+    - funlen # replaced by revive
+    - godox # TODO's are OK
+    - ireturn # It's OK
+    - musttag
+    - nonamedreturns
+    - goconst # finds repeated strings that could be replaced by a constant
+    - goheader # checks is file header matches to pattern
+    - gomodguard # [use more powerful depguard] allow and block lists linter for direct Go module dependencies
+    - gomoddirectives
+    - err113 # bad advice about dynamic errors
+    - lll # [replaced by golines] reports long lines
+    - mnd # detects magic numbers, duplicated by revive
+    - nlreturn # [too strict and mostly code is not more readable] checks for a new line before return and branch statements to increase code clarity
+    - noinlineerr # disallows inline error handling `if err := ...; err != nil {`
+    - prealloc # [premature optimization, but can be used in some cases] finds slice declarations that could potentially be preallocated
+    - tagliatelle # needs configuration
+    - testableexamples # checks if examples are testable (have an expected output)
+    - testpackage # makes you use a separate _test package
+    - paralleltest # not every test should be in parallel
+    - wrapcheck # not required
+    - wsl # [too strict and mostly code is not more readable] whitespace linter forces you to use empty lines
+    - wsl_v5 # [too strict and mostly code is not more readable] add or remove empty lines
+    - zerologlint # detects the wrong usage of zerolog that a user forgets to dispatch zerolog.Event
+
+  # All settings can be found here https://github.com/golangci/golangci-lint/blob/HEAD/.golangci.reference.yml
+  settings:
+    depguard:
+      rules:
+        "deprecated":
+          files:
+            - "$all"
+          deny:
+            - pkg: github.com/golang/protobuf
+              desc: Use google.golang.org/protobuf instead, see https://developers.google.com/protocol-buffers/docs/reference/go/faq#modules
+            - pkg: github.com/satori/go.uuid
+              desc: Use github.com/google/uuid instead, satori's package is not maintained
+            - pkg: github.com/gofrs/uuid$
+              desc: Use github.com/gofrs/uuid/v5 or later, it was not a go module before v5
+        "non-test files":
+          files:
+            - "!$test"
+          deny:
+            - pkg: math/rand$
+              desc: Use math/rand/v2 instead, see https://go.dev/blog/randv2
+            - pkg: "github.com/sirupsen/logrus"
+              desc: not allowed
+            - pkg: "github.com/pkg/errors"
+              desc: Should be replaced by standard lib errors package
+
+    dupl:
+      # token count (default: 150)
+      threshold: 300
+
+    embeddedstructfieldcheck:
+      # Checks that sync.Mutex and sync.RWMutex are not used as embedded fields.
+      forbid-mutex: true
+
+    errcheck:
+      # Report about not checking of errors in type assertions: `a := b.(MyStruct)`.
+      check-type-assertions: true
+      check-blank: true
+
+    exhaustive:
+      # Program elements to check for exhaustiveness.
+      # Default: [ switch ]
+      check:
+        - switch
+        - map
+      default-signifies-exhaustive: true
+
+    fatcontext:
+      # Check for potential fat contexts in struct pointers.
+      # May generate false positives.
+      # Default: false
+      check-struct-pointers: true
+
+    funcorder:
+      # Checks if the exported methods of a structure are placed before the non-exported ones.
+      struct-method: false
+
+    gocognit:
+      min-complexity: 55
+
+    gocritic:
+      enable-all: true
+      disabled-checks:
+        - paramTypeCombine
+      # The list of supported checkers can be found at https://go-critic.com/overview.
+      settings:
+        captLocal:
+          # Whether to restrict checker to params only.
+          paramsOnly: false
+        underef:
+          # Whether to skip (*x).method() calls where x is a pointer receiver.
+          skipRecvDeref: false
+        hugeParam:
+          # Default: 80
+          sizeThreshold: 200
+
+    govet:
+      enable-all: true
+
+    godot:
+      scope: toplevel
+
+    inamedparam:
+      # Skips check for interface methods with only a single parameter.
+      skip-single-param: true
+
+    nakedret:
+      # Default: 30
+      max-func-lines: 7
+
+    nestif:
+      min-complexity: 15
+
+    nolintlint:
+      # Exclude following linters from requiring an explanation.
+      # Default: []
+      allow-no-explanation: [funlen, gocognit, golines]
+      # Enable to require an explanation of nonzero length after each nolint directive.
+      require-explanation: true
+      # Enable to require nolint directives to mention the specific linter being suppressed.
+      require-specific: true
+
+    revive:
+      enable-all-rules: true
+      rules:
+        - name: add-constant
+          severity: warning
+          disabled: true
+        - name: cognitive-complexity
+          disabled: true # prefer maintidx
+        - name: cyclomatic
+          disabled: true # prefer maintidx
+        - name: function-length
+          arguments: [150, 225]
+        - name: line-length-limit
+          arguments: [150]
+        - name: nested-structs
+          disabled: true
+        - name: max-public-structs
+          arguments: [10]
+        - name: flag-parameter # fixes are difficult
+          disabled: true
+
+    rowserrcheck:
+      # database/sql is always checked.
+      # Default: []
+      packages:
+        - github.com/jmoiron/sqlx
+
+    perfsprint:
+      # optimize fmt.Sprintf("x: %s", y) into "x: " + y
+      strconcat: false
+
+    staticcheck:
+      checks:
+        - all
+
+    usetesting:
+      # Enable/disable `os.TempDir()` detections.
+      # Default: false
+      os-temp-dir: true
+
+    varnamelen:
+      max-distance: 75
+      min-name-length: 2
+      check-receivers: false
+      ignore-names:
+        - r
+        - w
+        - f
+        - err
+
+  exclusions:
+    # Default: []
+    presets:
+      - common-false-positives
+    rules:
+      # Allow "err" and "ok" vars to shadow existing declarations, otherwise we get too many false positives.
+      - text: '^shadow: declaration of "(err|ok)" shadows declaration'
+        linters:
+          - govet
+      - text: "parameter 'ctx' seems to be unused, consider removing or renaming it as _"
+        linters:
+          - revive
+      - path: _test\.go
+        linters:
+          - dupl
+          - gosec
+          - godot
+          - govet # alignment
+          - noctx
+          - perfsprint
+          - revive
+          - varnamelen

.yamllint

@@ -0,0 +1,16 @@
+---
+extends: default
+
+rules:
+  braces:
+    max-spaces-inside: 1
+  brackets:
+    max-spaces-inside: 1
+  comments: disable
+  comments-indentation: disable
+  document-start: disable
+  line-length:
+    level: warning
+    max: 160
+    allow-non-breakable-inline-mappings: true
+  truthy: disable

Makefile

@@ -0,0 +1,69 @@
+
+# BEGIN: lint-install .
+# http://github.com/codeGROOVE-dev/lint-install
+
+.PHONY: lint
+lint: _lint
+
+LINT_ARCH := $(shell uname -m)
+LINT_OS := $(shell uname)
+LINT_OS_LOWER := $(shell echo $(LINT_OS) | tr '[:upper:]' '[:lower:]')
+LINT_ROOT := $(shell dirname $(realpath $(firstword $(MAKEFILE_LIST))))
+
+# shellcheck and hadolint lack arm64 native binaries: rely on x86-64 emulation
+ifeq ($(LINT_OS),Darwin)
+	ifeq ($(LINT_ARCH),arm64)
+		LINT_ARCH=x86_64
+	endif
+endif
+
+LINTERS :=
+FIXERS :=
+
+GOLANGCI_LINT_CONFIG := $(LINT_ROOT)/.golangci.yml
+GOLANGCI_LINT_VERSION ?= v2.5.0
+GOLANGCI_LINT_BIN := $(LINT_ROOT)/out/linters/golangci-lint-$(GOLANGCI_LINT_VERSION)-$(LINT_ARCH)
+$(GOLANGCI_LINT_BIN):
+	mkdir -p $(LINT_ROOT)/out/linters
+	rm -rf $(LINT_ROOT)/out/linters/golangci-lint-*
+	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(LINT_ROOT)/out/linters $(GOLANGCI_LINT_VERSION)
+	mv $(LINT_ROOT)/out/linters/golangci-lint $@
+
+LINTERS += golangci-lint-lint
+golangci-lint-lint: $(GOLANGCI_LINT_BIN)
+	find . -name go.mod -execdir "$(GOLANGCI_LINT_BIN)" run -c "$(GOLANGCI_LINT_CONFIG)" \;
+
+FIXERS += golangci-lint-fix
+golangci-lint-fix: $(GOLANGCI_LINT_BIN)
+	find . -name go.mod -execdir "$(GOLANGCI_LINT_BIN)" run -c "$(GOLANGCI_LINT_CONFIG)" --fix \;
+
+YAMLLINT_VERSION ?= 1.37.1
+YAMLLINT_ROOT := $(LINT_ROOT)/out/linters/yamllint-$(YAMLLINT_VERSION)
+YAMLLINT_BIN := $(YAMLLINT_ROOT)/dist/bin/yamllint
+$(YAMLLINT_BIN):
+	mkdir -p $(LINT_ROOT)/out/linters
+	rm -rf $(LINT_ROOT)/out/linters/yamllint-*
+	curl -sSfL https://github.com/adrienverge/yamllint/archive/refs/tags/v$(YAMLLINT_VERSION).tar.gz | tar -C $(LINT_ROOT)/out/linters -zxf -
+	cd $(YAMLLINT_ROOT) && pip3 install --target dist . || pip install --target dist .
+
+LINTERS += yamllint-lint
+yamllint-lint: $(YAMLLINT_BIN)
+	PYTHONPATH=$(YAMLLINT_ROOT)/dist $(YAMLLINT_ROOT)/dist/bin/yamllint .
+
+.PHONY: _lint $(LINTERS)
+_lint:
+	@exit_code=0; \
+	for target in $(LINTERS); do \
+		$(MAKE) $$target || exit_code=1; \
+	done; \
+	exit $$exit_code
+
+.PHONY: fix $(FIXERS)
+fix:
+	@exit_code=0; \
+	for target in $(FIXERS); do \
+		$(MAKE) $$target || exit_code=1; \
+	done; \
+	exit $$exit_code
+
+# END: lint-install .

README.md

@@ -48,14 +48,12 @@ This model is based on early research combining COCOMO II effort estimation with
 
 ### Delay Costs
 
-**Project Delay (20%)**: Opportunity cost of blocked engineer time: `hourly_rate × duration_hours × 0.20`.
+**Project Delay (20%)**: Opportunity cost of blocked engineer time: `hourly_rate × duration_hours × 0.20`. Capped at 60 days (2 months).
 
-**Code Updates**: Rework cost from code drift. Power-law formula: `driftMultiplier = 1 + (0.03 × days^0.7)`, calibrated to 4% weekly churn. Applies to PRs open 3+ days, capped at 90 days. Based on Windows Vista analysis (Nagappan et al., Microsoft Research, 2008).
+**Code Updates**: Rework cost from code drift. Probability-based formula: `drift = 1 - (0.96)^(days/7)`, modeling the cumulative probability that code becomes stale with 4% weekly churn. Applies to PRs open 3+ days, capped at 90 days (~41% max drift). Based on Windows Vista analysis (Nagappan et al., Microsoft Research, 2008).
 
 **Future GitHub**: Cost for 3 future events (push, review, merge) with full context switching.
 
-External contributors (no write access) receive 50% delay cost reduction.
-
 ## Session-Based Time Tracking
 
 Events within 60 minutes are grouped into sessions to model real work patterns. This preserves flow state during continuous work and applies context switching costs only when work is interrupted.

cmd/prcost/main.go

@@ -54,7 +54,7 @@ func main() {
 	cfg := cost.DefaultConfig()
 	cfg.AnnualSalary = *salary
 	cfg.BenefitsMultiplier = *benefits
-	cfg.MinutesPerEvent = *eventMinutes
+	cfg.EventDuration = time.Duration(*eventMinutes) * time.Minute
 	cfg.DelayCostFactor = *overheadFactor
 
 	// Get GitHub token from gh CLI
@@ -152,20 +152,21 @@ func printHumanReadable(b cost.Breakdown, prURL string) {
 	// Delay Cost
 	fmt.Printf("DELAY COST\n")
 	if b.DelayCapped {
-		fmt.Printf("  Project Delay (20%%)         $%10.2f   (%.0f hrs, capped at 90 days)\n",
-			b.DelayCostDetail.ProjectDelayCost, b.DelayCostDetail.ProjectDelayHours)
+		fmt.Printf("  %-32s $%10.2f   (%.0f hrs, capped at 60 days)\n",
+			"Project Delay (20%)", b.DelayCostDetail.ProjectDelayCost, b.DelayCostDetail.ProjectDelayHours)
 	} else {
-		fmt.Printf("  Project Delay (20%%)         $%10.2f   (%.2f hrs)\n",
-			b.DelayCostDetail.ProjectDelayCost, b.DelayCostDetail.ProjectDelayHours)
+		fmt.Printf("  %-32s $%10.2f   (%.2f hrs)\n",
+			"Project Delay (20%)", b.DelayCostDetail.ProjectDelayCost, b.DelayCostDetail.ProjectDelayHours)
 	}
 
 	if b.DelayCostDetail.ReworkPercentage > 0 {
-		fmt.Printf("  Code Updates (%.0f%% rework)  $%10.2f   (%.2f hrs)\n",
-			b.DelayCostDetail.ReworkPercentage*100, b.DelayCostDetail.CodeUpdatesCost, b.DelayCostDetail.CodeUpdatesHours)
+		label := fmt.Sprintf("Code Updates (%.0f%% rework)", b.DelayCostDetail.ReworkPercentage)
+		fmt.Printf("  %-32s $%10.2f   (%.2f hrs)\n",
+			label, b.DelayCostDetail.CodeUpdatesCost, b.DelayCostDetail.CodeUpdatesHours)
 	}
 
-	fmt.Printf("  Future GitHub (3 events)    $%10.2f   (%.2f hrs)\n",
-		b.DelayCostDetail.FutureGitHubCost, b.DelayCostDetail.FutureGitHubHours)
+	fmt.Printf("  %-32s $%10.2f   (%.2f hrs)\n",
+		"Future GitHub (3 events)", b.DelayCostDetail.FutureGitHubCost, b.DelayCostDetail.FutureGitHubHours)
 	fmt.Printf("  ---\n")
 
 	if b.DelayCapped {