MediaWiki:Gadget-FormattingFixer.js - Revision history

Maintenance script: Change WikiEditor icons to articleCheck and code

2026-01-06T01:16:49Z

Change WikiEditor icons to articleCheck and code

← Older revision		Revision as of 01:16, 6 January 2026
Line 1,531:		Line 1,531:
	label: 'Fix Citations',		label: 'Fix Citations',
	type: 'button',		type: 'button',
	oouiIcon: '~~check~~',		oouiIcon: 'articleCheck',
	action: {		action: {
	type: 'callback',		type: 'callback',
Line 1,561:		Line 1,561:
	label: 'Fix Citations + Auto Add Links',		label: 'Fix Citations + Auto Add Links',
	type: 'button',		type: 'button',
	oouiIcon: '~~check~~',		oouiIcon: 'code',
	action: {		action: {
	type: 'callback',		type: 'callback',

Maintenance script: Remove VisualEditor integration (Parsoid issues), change Fix All icon to check

2026-01-06T00:26:08Z

Remove VisualEditor integration (Parsoid issues), change Fix All icon to check

← Older revision		Revision as of 00:26, 6 January 2026
Line 1,561:		Line 1,561:
	label: 'Fix Citations + Auto Add Links',		label: 'Fix Citations + Auto Add Links',
	type: 'button',		type: 'button',
	oouiIcon: '~~wikiText~~',		oouiIcon: 'check',
	action: {		action: {
	type: 'callback',		type: 'callback',
Line 1,667:		Line 1,667:
	function init() {		function init() {
	var action = mw.config.get('wgAction');		var action = mw.config.get('wgAction');
	~~var veLoaded = mw.config.get('wgVisualEditor');~~

	console.log('FormattingFixer: Initializing, action=' + action);		console.log('FormattingFixer: Initializing, action=' + action);

	// ~~For~~ VisualEditor		// VisualEditor integration removed - Parsoid causes too many issues with references and formatting
	~~if (typeof ve !== 'undefined' \|\| (veLoaded && veLoaded.pageLanguageDir)) {~~
	~~console.log('FormattingFixer: Setting up VisualEditor hooks');~~
	~~addVisualEditorButtons();~~
	}

	~~// Hook for when VE loads later~~
	~~mw.hook('ve.activationComplete').add(function() {~~
	~~console.log('FormattingFixer: VE activation complete');~~
	~~addVisualEditorButtons();~~
	~~});~~

	// For source editing (action=edit without VE)		// For source editing (action=edit without VE)
Line 1,771:		Line 1,760:

	})();		})();
	// Force update timestamp: Mon Jan 5 18:52:11 EST 2026		// Force update timestamp: Mon Jan 5 19:24:00 EST 2026

Maintenance script: Chapters always linked (case-sensitive), characters first occurrence only

2026-01-06T00:14:09Z

Chapters always linked (case-sensitive), characters first occurrence only

@@ Line 470: / Line 470: @@
          var database = {
-             targets: {},      // canonical name -> true
+             targets: {},      // canonical name -> true (characters, locations)
              aliases: {},      // alias -> canonical name
              displayText: {}   // search term -> { target: canonical, display: text }
@@ Line 528: / Line 529: @@
                              var content = page.revisions[0].slots.main['*'];
                              // Parse {{Chapter|number=X|title=Y|...}} entries
                              var chapterPattern = /\{\{Chapter\|[^}]*title=([^|}]+)/gi;
                              var match;
                              while ((match = chapterPattern.exec(content)) !== null) {
                                  var title = match[1].trim();
-                                 database.targets[title] = true;
+                                 database.chapters[title] = true;
+                             }
+                         }
@@ Line 775: / Line 778: @@
              return /^={2,}[^=].*={2,}\s*$/.test(line);
+         }
          // Third pass: Add links for unlinked terms (first occurrence only, respecting exclusions)
@@ Line 832: / Line 877: @@
          });
-         // Fourth pass: Remove duplicate links (keep first, remove subsequent) - but NOT in headers or See also
+         // Fourth pass: Remove duplicate links (keep first, remove subsequent) - but NOT in headers, See also, or chapters
          var seenLinks = {};
          var dupPattern = /\[\[([^\]|]+)(\|([^\]]+))?\]\]/g;
@@ Line 856: / Line 901: @@
              if (isInSeeAlsoSection(wikitext, absPos)) {
                  seenLinks[canonical] = true;
                  continue;
+             }

Maintenance script: Fix: Use line-based header detection instead of position-based (fixes header links being stripped)

2026-01-06T00:04:54Z

Fix: Use line-based header detection instead of position-based (fixes header links being stripped)

← Older revision		Revision as of 00:04, 6 January 2026
Line 735:		Line 735:
	});		});

	~~// LOGGING: Track decision path for specific headers (debug-20250105-2)~~
	~~if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {~~
	~~console.log('DEBUG: Processing header "' + title + '" (Level ' + level + ')');~~
	~~console.log('DEBUG: Stack before push: ' + sectionStack.map(function(s) { return s.title + '(' + s.level + ')'; }).join(' > '));~~
	~~console.log('DEBUG: isRelationships=' + isRelationships);~~
	}

	if (isRelationships) {		if (isRelationships) {
	// Check if title is a known target/alias.		// Check if title is a known target/alias.
Line 747:		Line 740:
	var titleKey = title.toLowerCase();		var titleKey = title.toLowerCase();
	var canonical = targetsByLower[titleKey] \|\| aliasesByLower[titleKey] \|\| null;		var canonical = targetsByLower[titleKey] \|\| aliasesByLower[titleKey] \|\| null;

	~~if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {~~
	~~console.log('DEBUG: Lookup "' + titleKey + '" -> ' + canonical);~~
	}

	// Only link if known target/alias and not already linked.		// Only link if known target/alias and not already linked.
Line 757:		Line 746:
	line = headerMatch[1] + ' [[' + canonical + ']] ' + headerMatch[1];		line = headerMatch[1] + ' [[' + canonical + ']] ' + headerMatch[1];
	fixes.push('Linked "' + canonical + '" in Relationships header');		fixes.push('Linked "' + canonical + '" in Relationships header');

	~~if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {~~
	~~console.log('DEBUG: Changed to ' + line);~~
	}
	}		}
	}		}
Line 778:		Line 763:
	* ensuring the first occurrence is always the one that is linked.		* ensuring the first occurrence is always the one that is linked.
	*/		*/

			// Helper: Check if an index in body is on a header line (line-based, not position-based)
			// This is more reliable than isInsideSectionHeader after body has been modified
			function isOnHeaderLine(text, idx) {
			// Find the line containing this index
			var lineStart = text.lastIndexOf('\n', idx - 1) + 1;
			var lineEnd = text.indexOf('\n', idx);
			if (lineEnd === -1) lineEnd = text.length;
			var line = text.substring(lineStart, lineEnd);
			// Check if this line is a section header (== ... ==)
			return /^={2,}[^=].={2,}\s$/.test(line);
			}

	// Third pass: Add links for unlinked terms (first occurrence only, respecting exclusions)		// Third pass: Add links for unlinked terms (first occurrence only, respecting exclusions)
Line 795:		Line 792:

	while ((termMatch = termPattern.exec(body)) !== null) {		while ((termMatch = termPattern.exec(body)) !== null) {
			// Skip if on a header line (use line-based detection, not position-based)
			if (isOnHeaderLine(body, termMatch.index)) continue;

			// Skip if in See also section (position-based is OK here since wikitext hasn't changed)
	var absPos = firstSectionIdx + termMatch.index;		var absPos = firstSectionIdx + termMatch.index;

	~~// Skip if inside a section header~~
	~~if (isInsideSectionHeader(wikitext, absPos)) continue;~~

	~~// Skip if in See also section~~
	if (isInSeeAlsoSection(wikitext, absPos)) continue;		if (isInSeeAlsoSection(wikitext, absPos)) continue;

Line 844:		Line 840:

	while ((dupMatch = dupPattern.exec(body)) !== null) {		while ((dupMatch = dupPattern.exec(body)) !== null) {
	~~var absPos = firstSectionIdx + dupMatch.index;~~
	var target = dupMatch[1].trim();		var target = dupMatch[1].trim();
	var canonical = database.aliases[target] \|\| target;		var canonical = database.aliases[target] \|\| target;
Line 851:		Line 846:
	// Header links (e.g., === [[Augustus]] ===) are independent from body links.		// Header links (e.g., === [[Augustus]] ===) are independent from body links.
	// The first body occurrence should still be linked even if a header link exists.		// The first body occurrence should still be linked even if a header link exists.
	if (~~isInsideSectionHeader~~(~~wikitext~~, ~~absPos~~)) {		// Use line-based detection since body has been modified.
			if (isOnHeaderLine(body, dupMatch.index)) {
	continue;		continue;
	}		}
Line 857:		Line 853:
	// Don't touch links in See also, but DO mark them as seen.		// Don't touch links in See also, but DO mark them as seen.
	// If a name appears in See also, we don't want to link it again in the body.		// If a name appears in See also, we don't want to link it again in the body.
			var absPos = firstSectionIdx + dupMatch.index;
	if (isInSeeAlsoSection(wikitext, absPos)) {		if (isInSeeAlsoSection(wikitext, absPos)) {
	seenLinks[canonical] = true;		seenLinks[canonical] = true;

Maintenance script: Simplify Relationships regex and force update with debug logging

2026-01-05T23:54:17Z

Simplify Relationships regex and force update with debug logging

← Older revision		Revision as of 23:54, 5 January 2026
Line 728:		Line 728:

	// Check if we are inside a Relationships section hierarchy		// Check if we are inside a Relationships section hierarchy
	// Scan up the stack to find if any ancestor is a "Relationships" section		// Scan up the stack to find if any ancestor is a "Relationships" section.
			// Simplified regex to match "Relationships", "Major Relationships", "Minor Relationships" more robustly.
	var isRelationships = sectionStack.some(function(section) {		var isRelationships = sectionStack.some(function(section) {
	return /^(Major~~\s+~~)?~~[Rr]elationships$\|^Minor~~\s~~+relationships~~$/i.test(section.title);		// Matches: "Relationships", "Major Relationships", "Minor Relationships" (case insensitive)
			return /^(Major\|Minor)?\s*Relationships$/i.test(section.title);
	});		});

	// LOGGING: Track decision path for specific headers (debug-20250105)		// LOGGING: Track decision path for specific headers (debug-20250105-2)
	if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {		if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {
	console.log('DEBUG: Processing header "' + title + '"');		console.log('DEBUG: Processing header "' + title + '" (Level ' + level + ')');
			console.log('DEBUG: Stack before push: ' + sectionStack.map(function(s) { return s.title + '(' + s.level + ')'; }).join(' > '));
	console.log('DEBUG: isRelationships=' + isRelationships);		console.log('DEBUG: isRelationships=' + isRelationships);
	~~console.log('DEBUG: Stack context: ' + sectionStack.map(function(s) { return s.title; }).join(' > '));~~
	}		}

Maintenance script: Add debug logging for Relationships headers (timestamped)

2026-01-05T23:52:17Z

Add debug logging for Relationships headers (timestamped)

← Older revision		Revision as of 23:52, 5 January 2026
Line 1,722:		Line 1,722:

	})();		})();
			// Force update timestamp: Mon Jan 5 18:52:11 EST 2026

Maintenance script: Debug Relationships header logic - Force Update

2026-01-05T23:49:43Z

Debug Relationships header logic - Force Update

← Older revision		Revision as of 23:49, 5 January 2026
Line 1:		Line 1:
	/**		/**
	* FormattingFixer for MediaWiki		* FormattingFixer for MediaWiki
	*		*
	* A comprehensive tool for standardizing citations and auto-linking content in wikitext.		* A comprehensive tool for standardizing citations and auto-linking content in wikitext.
	* Designed to work seamlessly with both VisualEditor (VE) and the classic WikiEditor.		* Designed to work seamlessly with both VisualEditor (VE) and the classic WikiEditor.
	*		*
	* KEY FEATURES:		* KEY FEATURES:
	*		*
	* 1. CITATION STANDARDIZATION		* 1. CITATION STANDARDIZATION

Maintenance script: Add debug logging with timestamp to force update

2026-01-05T23:47:44Z

Add debug logging with timestamp to force update

← Older revision		Revision as of 23:47, 5 January 2026
Line 733:		Line 733:
	});		});

	// LOGGING: Track decision path for specific headers		// LOGGING: Track decision path for specific headers (debug-20250105)
	if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {		if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {
	console.log('DEBUG: Processing header "' + title + '"');		console.log('DEBUG: Processing header "' + title + '"');

Maintenance script: Debug Relationships header logic

2026-01-05T23:46:03Z

Debug Relationships header logic

← Older revision		Revision as of 23:46, 5 January 2026
Line 733:		Line 733:
	});		});

			// LOGGING: Track decision path for specific headers
			if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {
			console.log('DEBUG: Processing header "' + title + '"');
			console.log('DEBUG: isRelationships=' + isRelationships);
			console.log('DEBUG: Stack context: ' + sectionStack.map(function(s) { return s.title; }).join(' > '));
			}

	if (isRelationships) {		if (isRelationships) {
	// Check if title is a known target/alias.		// Check if title is a known target/alias.
Line 738:		Line 745:
	var titleKey = title.toLowerCase();		var titleKey = title.toLowerCase();
	var canonical = targetsByLower[titleKey] \|\| aliasesByLower[titleKey] \|\| null;		var canonical = targetsByLower[titleKey] \|\| aliasesByLower[titleKey] \|\| null;

			if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {
			console.log('DEBUG: Lookup "' + titleKey + '" -> ' + canonical);
			}

	// Only link if known target/alias and not already linked.		// Only link if known target/alias and not already linked.
Line 744:		Line 755:
	line = headerMatch[1] + ' [[' + canonical + ']] ' + headerMatch[1];		line = headerMatch[1] + ' [[' + canonical + ']] ' + headerMatch[1];
	fixes.push('Linked "' + canonical + '" in Relationships header');		fixes.push('Linked "' + canonical + '" in Relationships header');

			if (/^(Jessica\|Daisy\|Tess\|Augustus)$/i.test(title)) {
			console.log('DEBUG: Changed to ' + line);
			}
	}		}
	}		}

Maintenance script: Fix Relationships header linkify: case-insensitive canonical lookup

2026-01-05T23:41:05Z

Fix Relationships header linkify: case-insensitive canonical lookup

← Older revision		Revision as of 23:41, 5 January 2026
Line 689:		Line 689:
	}		}
	allTerms.sort(function(a, b) { return b.term.length - a.term.length; });		allTerms.sort(function(a, b) { return b.term.length - a.term.length; });

			// Case-insensitive lookup tables for header linkification.
			// Some pages may have headings like "=== jessica ===" or extra whitespace.
			// We normalize to lowercase and resolve to the canonical page title.
			var targetsByLower = {};
			for (var t in database.targets) {
			if (database.targets.hasOwnProperty(t)) {
			targetsByLower[t.toLowerCase()] = t;
			}
			}
			var aliasesByLower = {};
			for (var a in database.aliases) {
			if (database.aliases.hasOwnProperty(a)) {
			aliasesByLower[a.toLowerCase()] = database.aliases[a];
			}
			}

	// First: Process section headers linearly to handle Relationships linking		// First: Process section headers linearly to handle Relationships linking
Line 718:		Line 734:

	if (isRelationships) {		if (isRelationships) {
	// Check if title is a target or alias		// Check if title is a known target/alias.
	var canonical = ~~database.targets~~[~~title~~] ~~? title : (database.aliases~~[~~title~~] \|\| null);		// Use case-insensitive lookup so "jessica" resolves to "Jessica".
			var titleKey = title.toLowerCase();
	// Only link if known target/alias and not already linked		var canonical = targetsByLower[titleKey] \|\| aliasesByLower[titleKey] \|\| null;
	if (canonical && !/\[\[/.test(line)) {
	line = headerMatch[1] + ' [[' + ~~title~~ + ']] ' + headerMatch[1];		// Only link if known target/alias and not already linked.
	fixes.push('Linked "' + ~~title~~ + '" in Relationships header');		// Use canonical title in the link to ensure proper capitalization.
	}		if (canonical && !/\[\[/.test(line)) {
			line = headerMatch[1] + ' [[' + canonical + ']] ' + headerMatch[1];
			fixes.push('Linked "' + canonical + '" in Relationships header');
			}
	}		}