';
- // Handle both old format (string array) and new format (object array)
+ // Migration function handles string array โ object array conversion on load
const skillFields = config.skillsSection.customFields || [];
skillFields.forEach((skill, index) => {
- // Support both old format (string) and new format (object)
- const skillName = typeof skill === 'string' ? skill : (skill.name || '');
- const skillDesc = typeof skill === 'string' ? '' : (skill.description || '');
- const skillEnabled = typeof skill === 'string' ? true : (skill.enabled !== false);
+ const skillName = skill.name || '';
+ const skillDesc = skill.description || '';
+ const skillEnabled = skill.enabled !== false;
html += `
@@ -518,9 +517,9 @@ function setupUserStatsListeners() {
});
// Toggle skill category
+ // Migration function handles string array โ object array conversion on load
$('.rpg-skill-toggle').off('change').on('change', function() {
const index = $(this).data('index');
- ensureSkillIsObject(index);
extensionSettings.trackerConfig.userStats.skillsSection.customFields[index].enabled = $(this).is(':checked');
saveSettings();
renderSkills();
@@ -529,7 +528,6 @@ function setupUserStatsListeners() {
// Rename skill category
$('.rpg-skill-name').off('blur').on('blur', function() {
const index = $(this).data('index');
- ensureSkillIsObject(index);
extensionSettings.trackerConfig.userStats.skillsSection.customFields[index].name = $(this).val();
saveSettings();
renderSkills();
@@ -538,27 +536,11 @@ function setupUserStatsListeners() {
// Update skill description
$('.rpg-skill-desc').off('blur').on('blur', function() {
const index = $(this).data('index');
- ensureSkillIsObject(index);
extensionSettings.trackerConfig.userStats.skillsSection.customFields[index].description = $(this).val();
saveSettings();
});
}
-/**
- * Helper to convert old string-format skill to object format
- */
-function ensureSkillIsObject(index) {
- const skill = extensionSettings.trackerConfig.userStats.skillsSection.customFields[index];
- if (typeof skill === 'string') {
- extensionSettings.trackerConfig.userStats.skillsSection.customFields[index] = {
- id: 'skill_' + Date.now(),
- name: skill,
- description: '',
- enabled: true
- };
- }
-}
-
/**
* Render Info Box configuration tab
*/
diff --git a/src/utils/avatars.js b/src/utils/avatars.js
index ed64a57..0b31e8f 100644
--- a/src/utils/avatars.js
+++ b/src/utils/avatars.js
@@ -14,9 +14,7 @@ import { getThumbnailUrl } from '../../../../../../script.js';
* @returns {string|null} Thumbnail URL or null if unavailable/error
*/
export function getSafeThumbnailUrl(type, filename) {
- // Return null if no filename provided
if (!filename || filename === 'none') {
- // console.log(`[RPG Companion] No valid filename provided for ${type} thumbnail`);
return null;
}
@@ -24,13 +22,11 @@ export function getSafeThumbnailUrl(type, filename) {
// Attempt to get thumbnail URL from SillyTavern API
const url = getThumbnailUrl(type, filename);
- // Validate that we got a string back
if (typeof url !== 'string' || url.trim() === '') {
console.warn(`[RPG Companion] getThumbnailUrl returned invalid result for ${type}:`, filename);
return null;
}
- // console.log(`[RPG Companion] Successfully generated ${type} thumbnail URL for: ${filename}`);
return url;
} catch (error) {
// Log detailed error information for debugging
diff --git a/src/utils/migration.js b/src/utils/migration.js
index b5b0f23..f04ca1f 100644
--- a/src/utils/migration.js
+++ b/src/utils/migration.js
@@ -27,9 +27,7 @@ const DEFAULT_INVENTORY_V2 = {
* @returns {MigrationResult} Migration result with v2 inventory and metadata
*/
export function migrateInventory(inventory) {
- // Case 1: Already v2 format (has version property and is an object)
if (inventory && typeof inventory === 'object' && inventory.version === 2) {
- // console.log('[RPG Companion Migration] Inventory already v2, no migration needed');
return {
inventory: inventory,
migrated: false,
@@ -37,9 +35,7 @@ export function migrateInventory(inventory) {
};
}
- // Case 2: null or undefined โ use defaults
if (inventory === null || inventory === undefined) {
- // console.log('[RPG Companion Migration] Inventory is null/undefined, using defaults');
return {
inventory: { ...DEFAULT_INVENTORY_V2 },
migrated: true,
@@ -47,12 +43,9 @@ export function migrateInventory(inventory) {
};
}
- // Case 3: v1 string format โ migrate to v2
if (typeof inventory === 'string') {
- // Check if it's an empty/default string
const trimmed = inventory.trim();
if (trimmed === '' || trimmed.toLowerCase() === 'none') {
- // console.log('[RPG Companion Migration] Inventory is empty/None, using defaults');
return {
inventory: { ...DEFAULT_INVENTORY_V2 },
migrated: true,
@@ -60,8 +53,6 @@ export function migrateInventory(inventory) {
};
}
- // Non-empty v1 string โ migrate to v2.onPerson
- // console.log('[RPG Companion Migration] Migrating v1 string to v2.onPerson:', inventory);
return {
inventory: {
version: 2,
diff --git a/style.css b/style.css
index c8379e7..27ab90e 100644
--- a/style.css
+++ b/style.css
@@ -1833,31 +1833,11 @@ body:has(.rpg-panel.rpg-position-left) #sheld {
border: 1px solid rgba(255, 255, 255, 0.1);
transition: all 0.2s ease;
width: 100%; /* Ensure cards take full width */
- max-height: clamp(200px, 18vh, 250px);
box-sizing: border-box; /* Include padding and border in width calculation */
flex-shrink: 0; /* Prevent cards from shrinking */
- overflow: hidden;
- scrollbar-width: thin;
- scrollbar-color: var(--rpg-border) transparent;
+ overflow: visible;
}
-.rpg-character-card::-webkit-scrollbar {
- width: 4px;
- height: 4px;
-}
-
-.rpg-character-card::-webkit-scrollbar-track {
- background: transparent;
-}
-
-.rpg-character-card::-webkit-scrollbar-thumb {
- background: var(--rpg-border);
- border-radius: 2px;
-}
-
-.rpg-character-card::-webkit-scrollbar-thumb:hover {
- background: var(--rpg-highlight);
-}
.rpg-character-card:hover {
background: rgba(0, 0, 0, 0.4);
@@ -1903,37 +1883,17 @@ body:has(.rpg-panel.rpg-position-left) #sheld {
display: flex;
flex-direction: column;
gap: 0;
- overflow: hidden;
+ overflow: visible;
}
.rpg-character-info {
display: flex;
flex-direction: column;
gap: clamp(3px, 0.5vh, 5px);
- overflow-y: auto;
- overflow-x: hidden;
+ overflow: visible;
flex: 1;
- min-height: 0;
- scrollbar-width: thin;
- scrollbar-color: var(--rpg-border) transparent;
}
-.rpg-character-info::-webkit-scrollbar {
- width: 4px;
-}
-
-.rpg-character-info::-webkit-scrollbar-track {
- background: transparent;
-}
-
-.rpg-character-info::-webkit-scrollbar-thumb {
- background: var(--rpg-border);
- border-radius: 2px;
-}
-
-.rpg-character-info::-webkit-scrollbar-thumb:hover {
- background: var(--rpg-highlight);
-}
/* Character header with emoji and name */
.rpg-character-header {
diff --git a/tests/integrationHelper.js b/tests/integrationHelper.js
deleted file mode 100644
index cef9307..0000000
--- a/tests/integrationHelper.js
+++ /dev/null
@@ -1,355 +0,0 @@
-/**
- * RPG Companion Integration Test Helper
- *
- * This module provides functions for testing JSON parsing and validation
- * within SillyTavern. It can be loaded in the browser console or integrated
- * into the extension's debug mode.
- *
- * Usage in browser console:
- * 1. Open SillyTavern with RPG Companion enabled
- * 2. Open browser dev tools (F12)
- * 3. Copy/paste this file's contents into console
- * 4. Run: RPGTestHelper.validateLastResponse() or other methods
- */
-
-window.RPGTestHelper = {
- /**
- * Validates the last generated tracker data against expected JSON structure
- */
- validateLastResponse() {
- console.log('๐ Validating last generated tracker data...\n');
-
- // Access extension settings (this assumes RPG Companion is loaded)
- const settings = window.extension_settings?.['rpg-companion'];
- if (!settings) {
- console.error('โ RPG Companion not found in extension_settings');
- return false;
- }
-
- const results = {
- inventoryV3: this.validateInventory(settings.inventoryV3),
- skillsV2: this.validateSkills(settings.skillsV2),
- questsV2: this.validateQuests(settings.questsV2),
- infoBoxData: this.validateInfoBox(settings.infoBoxData),
- charactersData: this.validateCharacters(settings.charactersData)
- };
-
- console.log('\n๐ Validation Results:');
- Object.entries(results).forEach(([key, valid]) => {
- console.log(` ${valid ? 'โ
' : 'โ'} ${key}`);
- });
-
- return Object.values(results).every(v => v);
- },
-
- /**
- * Validates inventory structure
- */
- validateInventory(inv) {
- console.log('\n๐ฆ Validating Inventory...');
-
- if (!inv) {
- console.log(' โ ๏ธ inventoryV3 is null/undefined');
- return true; // Not an error if not populated yet
- }
-
- let valid = true;
-
- // Check onPerson array
- if (inv.onPerson && !Array.isArray(inv.onPerson)) {
- console.log(' โ onPerson should be an array');
- valid = false;
- } else if (inv.onPerson?.length > 0) {
- const item = inv.onPerson[0];
- if (typeof item !== 'object' || !item.name) {
- console.log(' โ onPerson items should be objects with name property');
- valid = false;
- } else {
- console.log(` โ
onPerson: ${inv.onPerson.length} items (e.g., "${item.name}")`);
- }
- }
-
- // Check stored object
- if (inv.stored && typeof inv.stored !== 'object') {
- console.log(' โ stored should be an object');
- valid = false;
- } else if (inv.stored) {
- const locations = Object.keys(inv.stored);
- console.log(` โ
stored: ${locations.length} locations`);
- }
-
- // Check assets array
- if (inv.assets && !Array.isArray(inv.assets)) {
- console.log(' โ assets should be an array');
- valid = false;
- } else if (inv.assets?.length > 0) {
- console.log(` โ
assets: ${inv.assets.length} items`);
- }
-
- // Check simplified array
- if (inv.simplified && !Array.isArray(inv.simplified)) {
- console.log(' โ simplified should be an array');
- valid = false;
- } else if (inv.simplified?.length > 0) {
- console.log(` โ
simplified: ${inv.simplified.length} items`);
- }
-
- return valid;
- },
-
- /**
- * Validates skills structure
- */
- validateSkills(skills) {
- console.log('\nโ๏ธ Validating Skills...');
-
- if (!skills) {
- console.log(' โ ๏ธ skillsV2 is null/undefined');
- return true;
- }
-
- if (typeof skills !== 'object') {
- console.log(' โ skillsV2 should be an object');
- return false;
- }
-
- let valid = true;
-
- for (const [category, abilities] of Object.entries(skills)) {
- if (!Array.isArray(abilities)) {
- console.log(` โ ${category} should be an array`);
- valid = false;
- continue;
- }
-
- abilities.forEach((ability, i) => {
- if (typeof ability !== 'object' || !ability.name) {
- console.log(` โ ${category}[${i}] should be an object with name`);
- valid = false;
- }
- });
-
- console.log(` โ
${category}: ${abilities.length} abilities`);
- }
-
- return valid;
- },
-
- /**
- * Validates quests structure
- */
- validateQuests(quests) {
- console.log('\n๐ Validating Quests...');
-
- if (!quests) {
- console.log(' โ ๏ธ questsV2 is null/undefined');
- return true;
- }
-
- let valid = true;
-
- if (quests.main !== null && quests.main !== undefined) {
- if (typeof quests.main === 'string') {
- console.log(` โ
main: "${quests.main}"`);
- } else if (typeof quests.main === 'object' && quests.main.name) {
- console.log(` โ
main: "${quests.main.name}" (structured)`);
- } else {
- console.log(' โ main should be string or {name, description}');
- valid = false;
- }
- }
-
- if (quests.optional) {
- if (!Array.isArray(quests.optional)) {
- console.log(' โ optional should be an array');
- valid = false;
- } else {
- console.log(` โ
optional: ${quests.optional.length} quests`);
- }
- }
-
- return valid;
- },
-
- /**
- * Validates info box structure
- */
- validateInfoBox(info) {
- console.log('\n๐ Validating Info Box...');
-
- if (!info) {
- console.log(' โ ๏ธ infoBoxData is null/undefined');
- return true;
- }
-
- const fields = ['date', 'weather', 'temperature', 'time', 'location'];
- let valid = true;
-
- fields.forEach(field => {
- if (info[field] !== undefined && info[field] !== null) {
- if (typeof info[field] !== 'string') {
- console.log(` โ ${field} should be a string`);
- valid = false;
- } else {
- console.log(` โ
${field}: "${info[field]}"`);
- }
- }
- });
-
- if (info.recentEvents) {
- if (!Array.isArray(info.recentEvents)) {
- console.log(' โ recentEvents should be an array');
- valid = false;
- } else {
- console.log(` โ
recentEvents: ${info.recentEvents.length} events`);
- }
- }
-
- return valid;
- },
-
- /**
- * Validates characters structure
- */
- validateCharacters(chars) {
- console.log('\n๐ฅ Validating Characters...');
-
- if (!chars) {
- console.log(' โ ๏ธ charactersData is null/undefined');
- return true;
- }
-
- if (!Array.isArray(chars)) {
- console.log(' โ charactersData should be an array');
- return false;
- }
-
- let valid = true;
-
- chars.forEach((char, i) => {
- if (typeof char !== 'object' || !char.name) {
- console.log(` โ character[${i}] should have name`);
- valid = false;
- } else {
- console.log(` โ
${char.name}: ${char.relationship || 'no relationship'}`);
- }
- });
-
- return valid;
- },
-
- /**
- * Tests JSON extraction from a raw response string
- */
- testJSONExtraction(responseText) {
- console.log('\n๐ฌ Testing JSON Extraction...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = responseText.match(jsonRegex);
-
- if (!match) {
- console.log('โ No JSON code block found');
- return null;
- }
-
- console.log('โ
Found JSON code block');
-
- try {
- const parsed = JSON.parse(match[1].trim());
- console.log('โ
JSON parsed successfully');
- console.log('๐ Structure:', Object.keys(parsed).join(', '));
- return parsed;
- } catch (e) {
- console.log('โ JSON parse failed:', e.message);
-
- // Try to fix common issues
- console.log('๐ง Attempting to fix JSON...');
- const fixed = match[1].trim()
- .replace(/,\s*}/g, '}')
- .replace(/,\s*]/g, ']');
-
- try {
- const fixedParsed = JSON.parse(fixed);
- console.log('โ
Fixed JSON parsed successfully');
- return fixedParsed;
- } catch (e2) {
- console.log('โ Could not fix JSON:', e2.message);
- return null;
- }
- }
- },
-
- /**
- * Simulates a full parse cycle with a sample response
- */
- simulateParseResponse(sampleResponse) {
- console.log('\n๐ Simulating Parse Response...\n');
-
- const parsed = this.testJSONExtraction(sampleResponse);
-
- if (parsed) {
- console.log('\n๐ Validating parsed structure:');
-
- if (parsed.userStats) {
- console.log(' โ
userStats present');
- }
- if (parsed.skills) {
- console.log(' โ
skills present');
- }
- if (parsed.inventory) {
- console.log(' โ
inventory present');
- }
- if (parsed.quests) {
- console.log(' โ
quests present');
- }
- if (parsed.infoBox) {
- console.log(' โ
infoBox present');
- }
- if (parsed.presentCharacters) {
- console.log(' โ
presentCharacters present');
- }
- }
-
- return parsed;
- },
-
- /**
- * Prints current extension settings for debugging
- */
- printCurrentState() {
- const settings = window.extension_settings?.['rpg-companion'];
- if (!settings) {
- console.error('RPG Companion not found');
- return;
- }
-
- console.log('๐ Current RPG Companion State:\n');
- console.log('inventoryV3:', JSON.stringify(settings.inventoryV3, null, 2));
- console.log('skillsV2:', JSON.stringify(settings.skillsV2, null, 2));
- console.log('questsV2:', JSON.stringify(settings.questsV2, null, 2));
- console.log('infoBoxData:', JSON.stringify(settings.infoBoxData, null, 2));
- console.log('charactersData:', JSON.stringify(settings.charactersData, null, 2));
- },
-
- /**
- * Help message
- */
- help() {
- console.log(`
-๐งช RPG Companion Test Helper Commands:
-
-RPGTestHelper.validateLastResponse() - Validate current structured data
-RPGTestHelper.testJSONExtraction(text) - Test JSON extraction from text
-RPGTestHelper.simulateParseResponse(text) - Full parse simulation
-RPGTestHelper.printCurrentState() - Print current extension state
-RPGTestHelper.help() - Show this help message
-
-Example:
- RPGTestHelper.validateLastResponse()
- `);
- }
-};
-
-// Print help on load
-console.log('๐งช RPG Companion Test Helper loaded. Run RPGTestHelper.help() for commands.');
-
diff --git a/tests/jsonFormat.test.js b/tests/jsonFormat.test.js
deleted file mode 100644
index 8ad0063..0000000
--- a/tests/jsonFormat.test.js
+++ /dev/null
@@ -1,504 +0,0 @@
-/**
- * JSON Format Tests for RPG Companion
- *
- * These tests can be run in two ways:
- * 1. In browser console: Copy/paste or load as module in SillyTavern
- * 2. Via Node.js: Run with `node tests/jsonFormat.test.js`
- *
- * Tests cover:
- * - JSON prompt generation
- * - JSON response parsing
- * - Data structure validation
- */
-
-// Mock SillyTavern context for Node.js testing
-const isBrowser = typeof window !== 'undefined';
-
-// Sample mock data for testing
-const mockTrackerConfig = {
- userStats: {
- customStats: [
- { id: 'health', name: 'Health', enabled: true },
- { id: 'energy', name: 'Energy', enabled: true }
- ],
- showRPGAttributes: true,
- rpgAttributes: [
- { id: 'str', name: 'Strength', enabled: true },
- { id: 'dex', name: 'Dexterity', enabled: true }
- ],
- statusSection: {
- enabled: true,
- showMoodEmoji: true,
- customFields: ['Conditions']
- },
- skillsSection: {
- enabled: true,
- customFields: ['Combat', 'Stealth', 'Magic']
- }
- },
- infoBox: {
- widgets: {
- date: { enabled: true },
- weather: { enabled: true },
- temperature: { enabled: true, unit: 'C' },
- time: { enabled: true },
- location: { enabled: true },
- recentEvents: { enabled: true }
- }
- },
- presentCharacters: {
- showEmoji: true,
- relationshipFields: ['Enemy', 'Neutral', 'Friend', 'Lover'],
- customFields: [
- { id: 'appearance', name: 'Appearance', enabled: true },
- { id: 'demeanor', name: 'Demeanor', enabled: true }
- ],
- thoughts: { enabled: true, name: 'Thoughts' },
- characterStats: {
- enabled: true,
- customStats: [
- { id: 'health', name: 'Health', enabled: true },
- { id: 'arousal', name: 'Arousal', enabled: true }
- ]
- }
- }
-};
-
-// Sample JSON responses for testing parser
-const sampleValidJSONResponse = `
-Here's an interesting development in the story...
-
-\`\`\`json
-{
- "userStats": {
- "health": 85,
- "energy": 60,
- "str": 14,
- "dex": 12,
- "status": {
- "mood": "๐",
- "conditions": "Well-rested"
- }
- },
- "skills": {
- "combat": [
- { "name": "Sword Fighting", "description": "Basic melee combat with swords", "linkedItem": "Iron Sword" },
- { "name": "Parry", "description": "Deflect incoming attacks" }
- ],
- "stealth": [
- { "name": "Sneak", "description": "Move quietly" }
- ],
- "magic": []
- },
- "inventory": {
- "onPerson": [
- { "name": "Iron Sword", "description": "A sturdy blade" },
- { "name": "Leather Armor", "description": "Basic protection" },
- { "name": "Health Potion", "description": "Restores 50 HP" }
- ],
- "stored": {
- "Backpack": [
- { "name": "Rope", "description": "50 feet of hemp rope" }
- ]
- },
- "assets": [
- { "name": "Small Cottage", "description": "A humble dwelling in the village" }
- ]
- },
- "quests": {
- "main": "Find the Lost Artifact",
- "optional": ["Gather herbs for the healer", "Clear the rat infestation"]
- },
- "infoBox": {
- "date": "15th of Sunstone, Year 1423",
- "weather": "โ๏ธ Sunny",
- "temperature": "22ยฐC",
- "time": "Midday",
- "location": "Village Square",
- "recentEvents": ["Met the village elder", "Bought supplies"]
- },
- "presentCharacters": [
- {
- "name": "Elena",
- "description": "A young healer with kind eyes",
- "emoji": "๐",
- "relationship": "Friend",
- "stats": { "health": 100, "arousal": 10 },
- "appearance": "Long brown hair, green robes",
- "demeanor": "Cheerful and helpful",
- "thoughts": "I hope they can help me find the rare herbs..."
- }
- ]
-}
-\`\`\`
-
-The village was bustling with activity...
-`;
-
-const sampleMalformedJSONResponse = `
-Some story text here...
-
-\`\`\`json
-{
- "userStats": {
- "health": 85,
- "energy": 60,
- },
- "inventory": {
- "onPerson": [
- { "name": "Sword", "description": "Sharp" }
- ]
- }
-}
-\`\`\`
-`;
-
-const sampleSimplifiedInventoryResponse = `
-\`\`\`json
-{
- "userStats": {
- "health": 75,
- "energy": 50
- },
- "inventory": {
- "simplified": [
- { "name": "Magic Staff", "description": "Channels arcane energy" },
- { "name": "Spell Book", "description": "Contains basic spells" },
- { "name": "Mana Potion", "description": "Restores magical energy" }
- ]
- }
-}
-\`\`\`
-`;
-
-// Test results accumulator
-const testResults = {
- passed: 0,
- failed: 0,
- errors: []
-};
-
-/**
- * Simple assertion helper
- */
-function assert(condition, message) {
- if (condition) {
- testResults.passed++;
- console.log(`โ
PASS: ${message}`);
- } else {
- testResults.failed++;
- testResults.errors.push(message);
- console.error(`โ FAIL: ${message}`);
- }
-}
-
-/**
- * Test JSON extraction from markdown code blocks
- */
-function testJSONExtraction() {
- console.log('\n๐ Testing JSON Extraction from Code Blocks...\n');
-
- // Test 1: Extract valid JSON from code block
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- assert(match !== null, 'Should find JSON code block in response');
-
- if (match) {
- let parsed;
- try {
- parsed = JSON.parse(match[1].trim());
- assert(true, 'Should parse extracted JSON successfully');
- assert(parsed.userStats !== undefined, 'Parsed JSON should have userStats');
- assert(parsed.inventory !== undefined, 'Parsed JSON should have inventory');
- assert(parsed.skills !== undefined, 'Parsed JSON should have skills');
- assert(parsed.quests !== undefined, 'Parsed JSON should have quests');
- assert(parsed.infoBox !== undefined, 'Parsed JSON should have infoBox');
- assert(parsed.presentCharacters !== undefined, 'Parsed JSON should have presentCharacters');
- } catch (e) {
- assert(false, `Should not throw parsing error: ${e.message}`);
- }
- }
-
- // Test 2: Handle malformed JSON (trailing comma)
- const malformedMatch = sampleMalformedJSONResponse.match(jsonRegex);
- assert(malformedMatch !== null, 'Should find malformed JSON code block');
-
- if (malformedMatch) {
- try {
- JSON.parse(malformedMatch[1].trim());
- assert(false, 'Malformed JSON should throw parsing error');
- } catch (e) {
- assert(true, 'Malformed JSON correctly throws parsing error');
-
- // Test JSON fixing (remove trailing commas)
- const fixed = malformedMatch[1].trim()
- .replace(/,\s*}/g, '}')
- .replace(/,\s*]/g, ']');
- try {
- const fixedParsed = JSON.parse(fixed);
- assert(fixedParsed.userStats.health === 85, 'Fixed JSON should parse correctly');
- } catch (e2) {
- assert(false, `JSON fixing should work: ${e2.message}`);
- }
- }
- }
-}
-
-/**
- * Test inventory data structure validation
- */
-function testInventoryStructure() {
- console.log('\n๐ฆ Testing Inventory Structure...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- const data = JSON.parse(match[1].trim());
-
- const inv = data.inventory;
-
- // Test onPerson array structure
- assert(Array.isArray(inv.onPerson), 'onPerson should be an array');
- assert(inv.onPerson.length > 0, 'onPerson should have items');
- assert(inv.onPerson[0].name !== undefined, 'Items should have name property');
- assert(inv.onPerson[0].description !== undefined, 'Items should have description property');
-
- // Test stored object structure
- assert(typeof inv.stored === 'object', 'stored should be an object');
- assert(inv.stored.Backpack !== undefined, 'stored should have location keys');
- assert(Array.isArray(inv.stored.Backpack), 'stored locations should be arrays');
-
- // Test assets array
- assert(Array.isArray(inv.assets), 'assets should be an array');
-
- // Test simplified inventory
- const simplifiedMatch = sampleSimplifiedInventoryResponse.match(jsonRegex);
- const simplifiedData = JSON.parse(simplifiedMatch[1].trim());
- assert(Array.isArray(simplifiedData.inventory.simplified), 'simplified inventory should be an array');
-}
-
-/**
- * Test skills data structure validation
- */
-function testSkillsStructure() {
- console.log('\nโ๏ธ Testing Skills Structure...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- const data = JSON.parse(match[1].trim());
-
- const skills = data.skills;
-
- // Test skill categories
- assert(typeof skills === 'object', 'skills should be an object');
- assert(skills.combat !== undefined, 'skills should have combat category');
- assert(Array.isArray(skills.combat), 'skill categories should be arrays');
-
- // Test ability structure
- const ability = skills.combat[0];
- assert(ability.name !== undefined, 'Abilities should have name');
- assert(ability.description !== undefined, 'Abilities should have description');
-
- // Test linked item
- assert(ability.linkedItem === 'Iron Sword', 'First combat ability should be linked to Iron Sword');
- assert(skills.combat[1].linkedItem === undefined || skills.combat[1].linkedItem === null,
- 'Second combat ability should not have linkedItem');
-}
-
-/**
- * Test quests data structure
- */
-function testQuestsStructure() {
- console.log('\n๐ Testing Quests Structure...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- const data = JSON.parse(match[1].trim());
-
- const quests = data.quests;
-
- assert(typeof quests.main === 'string', 'main quest should be a string');
- assert(Array.isArray(quests.optional), 'optional quests should be an array');
- assert(quests.optional.length === 2, 'Should have 2 optional quests');
-}
-
-/**
- * Test characters data structure
- */
-function testCharactersStructure() {
- console.log('\n๐ฅ Testing Characters Structure...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- const data = JSON.parse(match[1].trim());
-
- const chars = data.presentCharacters;
-
- assert(Array.isArray(chars), 'presentCharacters should be an array');
- assert(chars.length > 0, 'Should have at least one character');
-
- const char = chars[0];
- assert(char.name === 'Elena', 'Character should have name');
- assert(char.description !== undefined, 'Character should have description');
- assert(char.emoji !== undefined, 'Character should have emoji');
- assert(char.relationship !== undefined, 'Character should have relationship');
- assert(char.stats !== undefined, 'Character should have stats');
- assert(char.thoughts !== undefined, 'Character should have thoughts');
-}
-
-/**
- * Test info box data structure
- */
-function testInfoBoxStructure() {
- console.log('\n๐ Testing Info Box Structure...\n');
-
- const jsonRegex = /```(?:json)?\s*([\s\S]*?)```/i;
- const match = sampleValidJSONResponse.match(jsonRegex);
- const data = JSON.parse(match[1].trim());
-
- const info = data.infoBox;
-
- assert(typeof info.date === 'string', 'date should be a string');
- assert(typeof info.weather === 'string', 'weather should be a string');
- assert(typeof info.temperature === 'string', 'temperature should be a string');
- assert(typeof info.time === 'string', 'time should be a string');
- assert(typeof info.location === 'string', 'location should be a string');
- assert(Array.isArray(info.recentEvents), 'recentEvents should be an array');
-}
-
-/**
- * Test JSON prompt schema generation (mock)
- */
-function testPromptSchemaGeneration() {
- console.log('\n๐ Testing Prompt Schema Generation...\n');
-
- // This tests the expected schema structure that generateJSONTrackerInstructions should produce
- const expectedSchemaProperties = [
- 'userStats',
- 'skills',
- 'inventory',
- 'quests',
- 'infoBox',
- 'presentCharacters'
- ];
-
- // Mock schema generation based on config
- const schema = {
- type: 'object',
- properties: {}
- };
-
- // User stats
- if (mockTrackerConfig.userStats) {
- schema.properties.userStats = { type: 'object', properties: {} };
-
- // Custom stats
- mockTrackerConfig.userStats.customStats.forEach(stat => {
- if (stat.enabled) {
- schema.properties.userStats.properties[stat.id] = {
- type: 'integer',
- minimum: 0,
- maximum: 100
- };
- }
- });
-
- // RPG attributes
- if (mockTrackerConfig.userStats.showRPGAttributes) {
- mockTrackerConfig.userStats.rpgAttributes.forEach(attr => {
- if (attr.enabled) {
- schema.properties.userStats.properties[attr.id] = {
- type: 'integer',
- minimum: 1
- };
- }
- });
- }
- }
-
- // Skills
- if (mockTrackerConfig.userStats.skillsSection?.enabled) {
- schema.properties.skills = { type: 'object', properties: {} };
- mockTrackerConfig.userStats.skillsSection.customFields.forEach(field => {
- const fieldId = field.toLowerCase();
- schema.properties.skills.properties[fieldId] = {
- type: 'array',
- items: {
- type: 'object',
- properties: {
- name: { type: 'string' },
- description: { type: 'string' },
- linkedItem: { type: 'string', nullable: true }
- }
- }
- };
- });
- }
-
- // Inventory
- schema.properties.inventory = {
- type: 'object',
- properties: {
- onPerson: { type: 'array' },
- stored: { type: 'object' },
- assets: { type: 'array' }
- }
- };
-
- // Validate schema structure
- assert(schema.properties.userStats !== undefined, 'Schema should include userStats');
- assert(schema.properties.userStats.properties.health !== undefined, 'Schema should include health stat');
- assert(schema.properties.skills !== undefined, 'Schema should include skills');
- assert(schema.properties.skills.properties.combat !== undefined, 'Schema should include combat skill');
- assert(schema.properties.inventory !== undefined, 'Schema should include inventory');
-
- console.log('Generated schema structure:', JSON.stringify(schema, null, 2).substring(0, 500) + '...');
-}
-
-/**
- * Run all tests
- */
-function runAllTests() {
- console.log('๐งช RPG Companion JSON Format Tests\n');
- console.log('='.repeat(50));
-
- try {
- testJSONExtraction();
- testInventoryStructure();
- testSkillsStructure();
- testQuestsStructure();
- testCharactersStructure();
- testInfoBoxStructure();
- testPromptSchemaGeneration();
- } catch (e) {
- console.error('๐ฅ Test suite error:', e);
- testResults.failed++;
- testResults.errors.push(e.message);
- }
-
- console.log('\n' + '='.repeat(50));
- console.log(`\n๐ Results: ${testResults.passed} passed, ${testResults.failed} failed`);
-
- if (testResults.errors.length > 0) {
- console.log('\nโ Failed tests:');
- testResults.errors.forEach(err => console.log(` - ${err}`));
- }
-
- return testResults;
-}
-
-// Export for module usage
-if (typeof module !== 'undefined' && module.exports) {
- module.exports = {
- runAllTests,
- sampleValidJSONResponse,
- sampleMalformedJSONResponse,
- sampleSimplifiedInventoryResponse,
- mockTrackerConfig
- };
-}
-
-// Auto-run if executed directly
-if (!isBrowser || (isBrowser && window.RPG_RUN_TESTS)) {
- runAllTests();
-}
-
diff --git a/tests/testRunner.html b/tests/testRunner.html
deleted file mode 100644
index fb6b2bd..0000000
--- a/tests/testRunner.html
+++ /dev/null
@@ -1,342 +0,0 @@
-
-
-
-
-
-
RPG Companion - Test Runner
-
-
-
-
-
RPG Companion Test Runner
-
-
-
This test suite validates the JSON format used for structured tracker data.
-
Tests cover: JSON extraction, parsing, data structure validation, and schema generation.
-
-
-
-
-
-
-
-
-
- 0
- Total
-
-
- 0
- Passed
-
-
- 0
- Failed
-
-
-
-
-
Test Output
-
- Click "Run All Tests" to start...
-
-
-
-
-
Valid JSON
-
Malformed JSON
-
Simplified Inventory
-
-
-
-
Sample Valid JSON Response
-
-
-
-
-
Sample Malformed JSON (with trailing comma)
-
-
-
-
-
Sample Simplified Inventory Response
-
-
-
-
- Note: These tests run in isolation and don't require SillyTavern to be running.
- For integration testing with actual LLM responses, use the Debug Mode in the extension settings.
-
-
-
-
-
-
-