ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-01-30 06:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: adress pr comments

Browse Source
This commit is contained in:
Shirone
2026-01-16 23:41:23 +01:00
parent b263cc615e
commit 50ed405c4a
2 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
apps/server/src/lib/xml-extractor.ts
View File

@@ -110,7 +110,7 @@ export function extractXmlElements(
const log = options.logger || logger;
const values: string[] = [];
const regex = new RegExp(`<${tagName}>(.*?)<\\/${tagName}>`, 'g');
const regex = new RegExp(`<${tagName}>([\\s\\S]*?)<\\/${tagName}>`, 'g');
const matches = xmlContent.matchAll(regex);
for (const match of matches) {
@@ -151,11 +151,11 @@ export function extractImplementedFeatures(
const featureContent = featureMatch[1];
// Extract name
const nameMatch = featureContent.match(/<name>(.*?)<\/name>/);
const nameMatch = featureContent.match(/<name>([\s\S]*?)<\/name>/);
const name = nameMatch ? unescapeXml(nameMatch[1].trim()) : '';
// Extract description
const descMatch = featureContent.match(/<description>(.*?)<\/description>/);
const descMatch = featureContent.match(/<description>([\s\S]*?)<\/description>/);
const description = descMatch ? unescapeXml(descMatch[1].trim()) : '';
// Extract file_locations if present
@@ -200,10 +200,9 @@ export function extractImplementedFeatureNames(
* @returns XML string for the feature
*/
export function featureToXml(feature: ImplementedFeature, indent: string = ' '): string {
const i1 = indent;
const i2 = indent + indent;
const i3 = indent + indent + indent;
const i4 = indent + indent + indent + indent;
const i2 = indent.repeat(2);
const i3 = indent.repeat(3);
const i4 = indent.repeat(4);
let xml = `${i2}<feature>
${i3}<name>${escapeXml(feature.name)}</name>

7
apps/server/tests/unit/lib/xml-extractor.test.ts
View File

@@ -702,10 +702,11 @@ describe('xml-extractor.ts', () => {
</item>
<item>second</item>
</items>`;
// Note: multiline content in single element may not be captured due to . not matching newlines
// Multiline content is now captured with [\s\S]*? pattern
const result = extractXmlElements(xml, 'item');
expect(result).toHaveLength(1); // Only matches single-line content
expect(result[0]).toBe('second');
expect(result).toHaveLength(2);
expect(result[0]).toBe('first');
expect(result[1]).toBe('second');
});
it('should handle consecutive elements without whitespace', () => {