pageCount drops the last partial page: pageCount(11, 5) returns 2 but should return 3. Please add a test.

Usage by agent

Timeline

1. planner iteration 1 ✓Finished 3 calls · 7,874 tokens · $0.0009 · 7.6s

   Loading…

2. coder iteration 1 ✓Finished 3 calls · 11.5K tokens · $0.0010 · 7.1s

   Loading…

3. tester iteration 1 ✓Finished 3 calls · 13.3K tokens · $0.0011 · 7.7s

   Loading…

4. verify iteration 1 ✓Passed 0.4s

   > buggy-node@1.0.0 test
   > node --test
   
   TAP version 13
   # Subtest: even division
   ok 1 - even division
     ---
     duration_ms: 0.947855
     type: 'test'
     ...
   # Subtest: partial last page
   ok 2 - partial last page
     ---
     duration_ms: 0.15294
     type: 'test'
     ...
   # Subtest: zero items
   ok 3 - zero items
     ---
     duration_ms: 0.104198
     type: 'test'
     ...
   # Subtest: single item needs one page
   ok 4 - single item needs one page
     ---
     duration_ms: 0.183655
     type: 'test'
     ...
   # Subtest: exact multiple does not add a page
   ok 5 - exact multiple does not add a page
     ---
     duration_ms: 0.120641
     type: 'test'
     ...
   1..5
   # tests 5
   # suites 0
   # pass 5
   # fail 0
   # cancelled 0
   # skipped 0
   # todo 0
   # duration_ms 83.685634
   

5. reviewer iteration 1 ✓Finished 1 calls · 4,128 tokens · $0.0007 · 3.1s

   Loading…

Final diff

diff --git a/pager.js b/pager.js
index 4f9c2b0..52510d8 100644
--- a/pager.js
+++ b/pager.js
@@ -1,5 +1,5 @@
 function pageCount(total, size) {
-  return Math.floor(total / size);
+  return Math.ceil(total / size);
 }
 
 module.exports = { pageCount };
diff --git a/pager.test.js b/pager.test.js
index f7a2181..fe2d014 100644
--- a/pager.test.js
+++ b/pager.test.js
@@ -5,3 +5,20 @@ const { pageCount } = require("./pager.js");
 test("even division", () => {
   assert.strictEqual(pageCount(10, 5), 2);
 });
+
+test("partial last page", () => {
+  assert.strictEqual(pageCount(11, 5), 3);
+});
+
+test("zero items", () => {
+  assert.strictEqual(pageCount(0, 5), 0);
+});
+
+test("single item needs one page", () => {
+  assert.strictEqual(pageCount(1, 5), 1);
+});
+
+test("exact multiple does not add a page", () => {
+  assert.strictEqual(pageCount(5, 5), 1);
+  assert.strictEqual(pageCount(6, 5), 2);
+});