cht-conf 6.1.0 → 6.1.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [6.1.1](https://github.com/medic/cht-conf/compare/v6.1.0...v6.1.1) (2026-04-10)
+
+
+### Bug Fixes
+
+* **#799:** batch contact IDs in move-contacts to avoid URI length errors ([#802](https://github.com/medic/cht-conf/issues/802)) ([08e92a5](https://github.com/medic/cht-conf/commit/08e92a568196bdcb3e68076a760fbd8a5b7ff461)), closes [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799) [#799](https://github.com/medic/cht-conf/issues/799)
+
 # [6.1.0](https://github.com/medic/cht-conf/compare/v6.0.2...v6.1.0) (2026-04-02)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cht-conf",
-  "version": "6.1.0",
+  "version": "6.1.1",
   "description": "Configure CHT deployments",
   "main": "./src/lib/main.js",
   "engines": {

package/src/lib/api.js

@@ -256,20 +256,23 @@ const api = {
     }
   },
 
-  async getReportsByCreatedByIds (createdByIds, limit, skip) {
+  async getReportsByFreetext (createdByIds, limit, bookmark) {
     const queryString = createdByIds.map(id => `exact_match:"contact:${id}"`).join(' OR ');
-    const res = await request.get(`${environment.apiUrl}/_design/medic/_nouveau/reports_by_freetext`, {
-      qs: {
-        // sorting by this field ensures that the output are same with the clouseau views
-        // https://github.com/medic/cht-core/pull/9541
-        sort: '"reported_date"',
-        q: queryString,
-        include_docs: true,
-        limit,
-        skip
-      }, json: true
+    const body = {
+      // sorting by this field ensures that the output are same with the clouseau views
+      // https://github.com/medic/cht-core/pull/9541
+      sort: 'reported_date',
+      q: queryString,
+      include_docs: true,
+      limit,
+    };
+    if (bookmark) {
+      body.bookmark = bookmark;
+    }
+    const res = await request.post(`${environment.apiUrl}/_design/medic/_nouveau/reports_by_freetext`, {
+      body, json: true
     });
-    return res.hits.map(item => item.doc);
+    return { docs: res.hits.map(item => item.doc), bookmark: res.bookmark };
   }
 };
 

package/src/lib/hierarchy-operations/delete-hierarchy.js

@@ -34,7 +34,7 @@ async function deleteReportsForContact(db, options, contact) {
   let skip = 0;
   let reportBatch;
   do {
-    reportBatch = await DataSource.getReportsForContacts(db, [], [contact._id], skip);
+    reportBatch = await DataSource.fetchReportsBySubject(db, [contact._id], skip);
 
     for (const report of reportBatch) {
       JsDocs.deleteDoc(options, report);

package/src/lib/hierarchy-operations/hierarchy-data-source.js

@@ -1,10 +1,11 @@
 const lineageManipulation = require('./lineage-manipulation');
-const {getValidApiVersion} = require('../get-api-version');
+const { getValidApiVersion } = require('../get-api-version');
 const semver = require('semver');
 const api = require('../api');
 
 const HIERARCHY_ROOT = 'root';
 const BATCH_SIZE = 10000;
+const QUERY_IDS_BATCH_SIZE = 100;
 const NOUVEAU_MIN_VERSION = '5.0.0';
 const SUBJECT_IDS = ['patient_id', 'patient_uuid', 'place_id', 'place_uuid'];
 
@@ -71,18 +72,39 @@ const getFromDbView = async (db, view, keys, skip) => {
   return res.rows.map(row => row.doc);
 };
 
-const fetchReportsByCreator = async (db, createdByIds, skip) => {
+const useNouveauSearch = async () => {
+  const coreVersion = await getValidApiVersion();
+  return coreVersion && semver.gte(coreVersion, NOUVEAU_MIN_VERSION);
+};
+
+const fetchReportsByCreator = async (db, createdByIds, cursor, useNouveau) => {
   if (createdByIds.length === 0) {
-    return [];
+    return { docs: [], cursor: null };
   }
 
-  const coreVersion = await getValidApiVersion();
-  if (coreVersion && semver.gte(coreVersion, NOUVEAU_MIN_VERSION)) {
-    return await api().getReportsByCreatedByIds(createdByIds, BATCH_SIZE, skip);
+  if (useNouveau) {
+    return await fetchAllReportsFromNouveau(createdByIds);
   }
 
+  const skip = cursor || 0;
   const createdByKeys = createdByIds.map(id => [`contact:${id}`]);
-  return await getFromDbView(db, 'medic-client/reports_by_freetext', createdByKeys, skip);
+  const docs = await getFromDbView(db, 'medic-client/reports_by_freetext', createdByKeys, skip);
+  return { docs, cursor: skip + docs.length };
+};
+
+const fetchAllReportsFromNouveau = async (createdByIds) => {
+  const allDocs = [];
+  for (let i = 0; i < createdByIds.length; i += QUERY_IDS_BATCH_SIZE) {
+    const idsBatch = createdByIds.slice(i, i + QUERY_IDS_BATCH_SIZE);
+    let bookmark = null;
+    let batch;
+    do {
+      batch = await api().getReportsByFreetext(idsBatch, BATCH_SIZE, bookmark);
+      allDocs.push(...batch.docs);
+      bookmark = batch.bookmark;
+    } while (batch.docs.length >= BATCH_SIZE);
+  }
+  return { docs: allDocs, cursor: null };
 };
 
 const fetchReportsBySubject = async (db, createdAtIds, skip) => {
@@ -92,18 +114,6 @@ const fetchReportsBySubject = async (db, createdAtIds, skip) => {
   return await getFromDbView(db, 'medic-client/reports_by_subject', createdAtIds, skip);
 };
 
-const getReportsForContacts = async (db, createdByIds, createdAtIds, skip) => {
-  const [creatorReports, subjectReports] = await Promise.all([
-    fetchReportsByCreator(db, createdByIds, skip),
-    fetchReportsBySubject(db, createdAtIds, skip)
-  ]);
-
-  const allRows = [...creatorReports, ...subjectReports];
-
-  const docsWithId = allRows.map(( doc ) => [doc._id, doc]);
-  return Array.from(new Map(docsWithId).values());
-};
-
 async function getAncestorsOf(db, contactDoc) {
   const ancestorIds = lineageManipulation.pluckIdsFromLineage(contactDoc.parent);
   const ancestors = await db.allDocs({
@@ -130,5 +140,7 @@ module.exports = {
   getContactWithDescendants,
   getContact,
   getContactsByIds,
-  getReportsForContacts,
+  useNouveauSearch,
+  fetchReportsByCreator,
+  fetchReportsBySubject,
 };

package/src/lib/hierarchy-operations/index.js

@@ -74,24 +74,39 @@ function getPrimaryContactId(doc) {
 
 async function updateReports(db, options, moveContext) {
   const descendantIds = moveContext.descendantsAndSelf.map(contact => contact._id);
+  const createdAtIds = getReportsCreatedAtIds(moveContext);
 
-  let skip = 0;
-  let reportDocsBatch;
-  do {
-    info(`Processing ${skip} to ${skip + DataSource.BATCH_SIZE} report docs`);
-    const createdAtIds = getReportsCreatedAtIds(moveContext);
-    reportDocsBatch = await DataSource.getReportsForContacts(db, descendantIds, createdAtIds, skip);
+  let totalCount = 0;
+  const useNouveau = await DataSource.useNouveauSearch();
 
-    const lineageUpdates = replaceLineageOfReportCreator(reportDocsBatch, moveContext);
-    const reassignUpdates = reassignReports(reportDocsBatch, moveContext);
-    const updatedReports = reportDocsBatch.filter(doc => lineageUpdates.has(doc._id) || reassignUpdates.has(doc._id));
-
-    minifyLineageAndWriteToDisk(options, updatedReports);
+  let cursor = null;
+  let result;
+  do {
+    info(`Processing creator reports ${totalCount} to ${totalCount + DataSource.BATCH_SIZE}`);
+    result = await DataSource.fetchReportsByCreator(db, descendantIds, cursor, useNouveau);
+    processAndWriteReportBatch(result.docs, options, moveContext);
+    cursor = result.cursor;
+    totalCount += result.docs.length;
+  } while (result.cursor && result.docs.length >= DataSource.BATCH_SIZE);
 
-    skip += reportDocsBatch.length;
-  } while (reportDocsBatch.length >= DataSource.BATCH_SIZE);
+  let skip = 0;
+  let batch;
+  do {
+    info(`Processing subject reports ${skip} to ${skip + DataSource.BATCH_SIZE}`);
+    batch = await DataSource.fetchReportsBySubject(db, createdAtIds, skip);
+    processAndWriteReportBatch(batch, options, moveContext);
+    skip += batch.length;
+    totalCount += batch.length;
+  } while (batch.length >= DataSource.BATCH_SIZE);
+
+  return totalCount;
+}
 
-  return skip;
+function processAndWriteReportBatch(batch, options, moveContext) {
+  const lineageUpdates = replaceLineageOfReportCreator(batch, moveContext);
+  const reassignUpdates = reassignReports(batch, moveContext);
+  const updatedReports = batch.filter(doc => lineageUpdates.has(doc._id) || reassignUpdates.has(doc._id));
+  minifyLineageAndWriteToDisk(options, updatedReports);
 }
 
 function getReportsCreatedAtIds(moveContext) {