nodebb-plugin-openai 1.3.0 → 1.3.2

package/eslint.config.mjs

@@ -0,0 +1,10 @@
+'use strict';
+
+import serverConfig from 'eslint-config-nodebb';
+import publicConfig from 'eslint-config-nodebb/public';
+
+export default [
+	...publicConfig,
+	...serverConfig,
+];
+

package/lib/summary.js

@@ -1,5 +1,6 @@
 'use strict';
 
+const db = require.main.require('./src/database');
 const batch = require.main.require('./src/batch');
 const topics = require.main.require('./src/topics');
 const posts = require.main.require('./src/posts');
@@ -7,7 +8,7 @@ const user = require.main.require('./src/user');
 
 function formatPosts(posts) {
 	return posts
-		.map((post) => `User ${post.username}:\n${post.content}`)
+		.map(post => `User ${post.username}:\n${post.content}`)
 		.join('\n---\n');
 }
 
@@ -16,7 +17,6 @@ function chunkPosts(posts, maxTokensPerChunk = 3000) {
 	let currentChunk = [];
 	let currentTokens = 0;
 
-	// eslint-disable-next-line no-restricted-syntax
 	for (const post of posts) {
 		const estimatedTokens = Math.ceil(post.content.length / 4) + 10;
 		if (currentTokens + estimatedTokens > maxTokensPerChunk) {
@@ -37,6 +37,9 @@ function chunkPosts(posts, maxTokensPerChunk = 3000) {
 
 async function summarizeChunk(chunk, openai, settings) {
 	const threadText = formatPosts(chunk);
+	if (!threadText) {
+		return '';
+	}
 	const response = await openai.chat.completions.create({
 		model: settings.model || 'gpt-3.5-turbo',
 		messages: [
@@ -55,7 +58,7 @@ async function summarizeChunk(chunk, openai, settings) {
 	return response.choices[0].message.content.trim();
 }
 
-exports.summarizeTopic = async function(tid, openai, settings) {
+exports.summarizeTopic = async function (tid, openai, settings) {
 	const allPids = await topics.getPids(tid);
 
 	const userMap = {};
@@ -63,12 +66,12 @@ exports.summarizeTopic = async function(tid, openai, settings) {
 	const chunkSummaries = [];
 
 	await batch.processArray(allPids, async function (pids) {
-		const postData = await posts.getPostsFields(pids, ['uid', 'content']);
-
+		let postData = await posts.getPostsFields(pids, ['uid', 'content', 'deleted']);
+		postData = postData.filter(p => !p.deleted);
 		const missingUids = [];
 		postData.forEach((p) => {
 			if (!userMap.hasOwnProperty(p.uid)) {
-				missingUids.push(p.uid)
+				missingUids.push(p.uid);
 			}
 		});
 		const userData = await user.getUsersFields(missingUids, ['username']);
@@ -82,7 +85,7 @@ exports.summarizeTopic = async function(tid, openai, settings) {
 		const chunks = chunkPosts(postData);
 
 		chunkSummaries.push(...await Promise.all(chunks.map(async (chunk) => {
-			return summarizeChunk(chunk, openai, settings)
+			return summarizeChunk(chunk, openai, settings);
 		})));
 	}, {
 		batch: 500,
@@ -92,7 +95,10 @@ exports.summarizeTopic = async function(tid, openai, settings) {
 		return chunkSummaries[0];
 	}
 	// Final summary from all summaries
-	const finalInput = chunkSummaries.join("\n\n");
+	const finalInput = chunkSummaries.join('\n\n');
+	if (!finalInput) {
+		return '';
+	}
 	const finalResponse = await openai.chat.completions.create({
 		model: settings.model || 'gpt-3.5-turbo',
 		messages: [
@@ -109,4 +115,8 @@ exports.summarizeTopic = async function(tid, openai, settings) {
 	});
 
 	return finalResponse.choices[0].message.content.trim();
+};
+
+exports.clearTopicSummary = async function (tids) {
+	await db.deleteObjectFields(tids.map(tid => `topic:${tid}`), ['openai:summary']);
 };

package/library.js

@@ -10,7 +10,6 @@ const controllers = require('./lib/controllers');
 const routeHelpers = require.main.require('./src/routes/helpers');
 const socketHelpers = require.main.require('./src/socket.io/helpers');
 const topics = require.main.require('./src/topics');
-const posts = require.main.require('./src/posts');
 const user = require.main.require('./src/user');
 const messaging = require.main.require('./src/messaging');
 const api = require.main.require('./src/api');
@@ -35,18 +34,19 @@ const defaults = {
 
 plugin.init = async (params) => {
 	const { router /* , middleware , controllers */ } = params;
-	const settings = await meta.settings.get('openai')
+	const settings = await meta.settings.get('openai');
 	if (settings && settings.apikey) {
 		openai = new OpenAI({
 			apiKey: settings.apikey,
 		});
+		plugin.openai = openai;
 	}
 
 	routeHelpers.setupAdminPageRoute(router, '/admin/plugins/openai', controllers.renderAdminPage);
 };
 
 async function getSettings() {
-	const settings = await meta.settings.get('openai')
+	const settings = await meta.settings.get('openai');
 	return {...defaults, ...settings };
 }
 
@@ -151,7 +151,7 @@ plugin.actionMessagingSave = async function (hookData) {
 			let messages = await messaging.getMessagesFields(mids, ['fromuid', 'content', 'system']);
 			messages = messages.filter(m => m && !m.system);
 			conversation = messages.map(
-				(msg) => ({ role: msg.fromuid === chatgptUid ? 'assistant' : 'user', content: msg.content })
+				msg => ({ role: msg.fromuid === chatgptUid ? 'assistant' : 'user', content: msg.content })
 			);
 		}
 
@@ -181,7 +181,8 @@ async function canUseOpenAI(uid, settings, silent = false) {
 
 async function checkReputation(uid, settings, silent) {
 	const reputation = await user.getUserField(uid, 'reputation');
-	const hasEnoughRep = parseInt(settings.minimumReputation, 10) === 0 || parseInt(reputation, 10) >= parseInt(settings.minimumReputation, 10);
+	const hasEnoughRep = parseInt(settings.minimumReputation, 10) === 0 ||
+		parseInt(reputation, 10) >= parseInt(settings.minimumReputation, 10);
 
 	if (!hasEnoughRep && !silent) {
 		sockets.server.in(`uid_${uid}`).emit('event:alert', {
@@ -277,6 +278,38 @@ plugin.filterTopicThreadTools = async (hookData) => {
 	return hookData;
 };
 
+plugin.actionTopicReply = async (hookData) => {
+	await summary.clearTopicSummary([hookData.post.tid]);
+};
+
+plugin.actionPostEdit = async (hookData) => {
+	await summary.clearTopicSummary([hookData.post.tid]);
+};
+
+plugin.actionPostsPurge = async (hookData) => {
+	const { posts } = hookData;
+	const uniqTids = [...new Set(posts.map(p => p.tid))];
+	await summary.clearTopicSummary(uniqTids);
+};
+
+plugin.actionPostRestore = async (hookData) => {
+	await summary.clearTopicSummary([hookData.post.tid]);
+};
+
+plugin.actionPostDelete = async (hookData) => {
+	await summary.clearTopicSummary([hookData.post.tid]);
+};
+
+plugin.actionPostMove = async (hookData) => {
+	await summary.clearTopicSummary([hookData.post.tid, hookData.tid]);
+};
+
+plugin.actionPostChangeOwner = async (hookData) => {
+	const { posts } = hookData;
+	const uniqTids = [...new Set(posts.map(p => p.tid))];
+	await summary.clearTopicSummary(uniqTids);
+};
+
 socketPlugins.openai = {};
 
 socketPlugins.openai.summarizeTopic = async function (socket, data) {
@@ -289,5 +322,11 @@ socketPlugins.openai.summarizeTopic = async function (socket, data) {
 		return;
 	}
 
-	return summary.summarizeTopic(tid, openai, settings);
+	let openaiSummary = await topics.getTopicField(tid, 'openai:summary');
+	if (openaiSummary) {
+		return openaiSummary;
+	}
+	openaiSummary = await summary.summarizeTopic(tid, openai, settings);
+	await topics.setTopicField(tid, 'openai:summary', openaiSummary);
+	return openaiSummary;
 };

package/package.json

@@ -1,12 +1,15 @@
 {
   "name": "nodebb-plugin-openai",
-  "version": "1.3.0",
+  "version": "1.3.2",
   "description": "A NodeBB OpenAI plugin",
   "main": "library.js",
   "repository": {
     "type": "git",
     "url": "https://github.com/nodebb/nodebb-plugin-openai"
   },
+  "scripts": {
+    "lint": "eslint ."
+  },
   "keywords": [
     "nodebb",
     "plugin",
@@ -30,17 +33,16 @@
   },
   "readmeFilename": "README.md",
   "nbbpm": {
-    "compatibility": "^3.2.0"
+    "compatibility": "^3.2.0 || ^4.0.0"
   },
   "dependencies": {
-    "openai": "4.53.2"
+    "openai": "4.98.0"
   },
   "devDependencies": {
     "@commitlint/cli": "19.3.0",
     "@commitlint/config-angular": "19.3.0",
-    "eslint": "8.57.0",
-    "eslint-config-nodebb": "0.2.1",
-    "eslint-plugin-import": "2.29.1",
+    "eslint": "9.26.0",
+    "eslint-config-nodebb": "^1.1.4",
     "husky": "9.0.11",
     "lint-staged": "15.2.7"
   }

package/plugin.json

@@ -7,7 +7,15 @@
 		{ "hook": "filter:admin.header.build", "method": "addAdminNavigation" },
 		{ "hook": "action:mentions.notify", "method": "actionMentionsNotify" },
 		{ "hook": "action:messaging.save", "method": "actionMessagingSave" },
-		{ "hook": "filter:topic.thread_tools", "method": "filterTopicThreadTools" }
+		{ "hook": "filter:topic.thread_tools", "method": "filterTopicThreadTools" },
+
+		{ "hook": "action:topic.reply", "method": "actionTopicReply" },
+		{ "hook": "action:post.edit", "method": "actionPostEdit" },
+		{ "hook": "action:posts.purge", "method": "actionPostsPurge" },
+		{ "hook": "action:post.restore", "method": "actionPostRestore" },
+ 		{ "hook": "action:post.delete", "method": "actionPostDelete" },
+		{ "hook": "action:post.move", "method": "actionPostMove" },
+		{ "hook": "action:post.changeOwner", "method": "actionPostChangeOwner" }
 	],
 	"scripts": [
 		"./public/lib/main.js"

package/templates/admin/plugins/openai.tpl

@@ -38,6 +38,8 @@
 							<option value="gpt-4o">gpt-4o</option>
 							<option value="gpt-4-turbo">gpt-4-turbo</option>
 							<option value="gpt-4">gpt-4</option>
+							<option value="gpt-4.1-mini">gpt-4.1-mini</option>
+							<option value="gpt-4.1">gpt-4.1</option>
 						</select>
 					</div>
 					<div class="">