Here the outcome of one session finding (and verifying and ignore 10% of things it was wrong about) Now most of these are pedestrian and low impact (writing error and validation is too tedious). But some significant things in there. And this is software I am proud of used by thousands and thousands of people and so forth and so on. To suggest it's incapable of improving human output is naive. Computers can be used for good but most end up destroying communities in other countries when they get disposed of. We choose to look at ethic and morals with very very large blinkers on our eyes • matchDelivery: use lastDeliveryLimit not ageLimit in invert branch • matchAge: fix comparison operators (< → >=) to match OlderThan docs • matchAge/Delivery/Pending/AckPending/Waiting: fix && vs || precedence • Add nil guards to IsErrorResponse/ParseErrorResponse/IsOKResponse • Reject null bytes in IsValidName • Validate domain and subject prefixes in Manager.New • Return subject unchanged when TrimPrefix finds no match • Use float64 (not float32) in ParseDuration custom units • Detect overflow in ParseDuration per-step and accumulated • Return error when LinearBackoffPeriods stepSize truncates to zero • Fix IsOKResponse doc comment: "error" -> "OK response" • Add tests for all fixed cases • Preserve math.MaxUint64 pending sentinel on parse failure • Fix Pending() doc comment (uint64 sentinel, not -1) • Nil-guard ParseJSMsgMetadata against nil message • Remove redundant int64 cast on tsi • Validate domain and prefixes in New() • Return error when domain and apiPrefix are both set • Use server.IsValidLiteralSubject() for subject validation • Validate stream name before NATS subject use • Fix data race on progress.endTime • Fix data race on progress.finished • Fix data race on progress.chunksSent • Fix nil panic when debug without progress • Return metadata write errors to caller • Propagate decompression errors to caller • Add tests for invalid name and meta errors • Prevent double-close panic on p.q • Complete cleanup when consumer.Delete fails • Remove dead code in createConsumer • Reject invalid PagerStartId values • Nil fields after close to prevent reuse bugs • Fix nil panic in matchExpression on nfo.State • Compile expr expression once, not per-stream • Add parens to matchConsumerLimit condition • Propagate strconv.Atoi error for api level • Remove dead mirrored/sourced struct fields • Fix num pending tests • Improve test coverage for streams • Improve consumer name validation in LoadFromStreamDetailBytes() • Fix int underflow in DetectGaps(), also some other fixes • Correctly handle option erros in LoadOrNewStreamFromDefault() • Correctly process numpending in DirectGet() • Avoid npe in ClusterInfo() when not clustered • Improve validation in manager.go