Enable GitAuto forge to call GitAuto with payload

[hiroshinishio]

No description provided.

[sourcery-ai]

Hey @hiroshinishio - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider removing commented-out code in getIssueDetails response object - if this data isn't needed, remove it entirely; if it might be needed later, add a TODO comment explaining why it's preserved

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟡 Security: 1 issue found
• 🟢 Testing: all looks good
• 🟡 Complexity: 1 issue found
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[sourcery-ai]

🚨 suggestion (security): Consider validating payload fields before sending to external service

Spreading the entire payload without validation could lead to sending unexpected data. Consider explicitly mapping required fields.

Suggested implementation:

    body: JSON.stringify({
      issueId: payload.issueId,
      repository: payload.repository,
    }),

For even better validation, you might want to add:

1. Type checking before the JSON.stringify:

if (!payload.issueId || !payload.repository) {
  throw new Error('Missing required fields: issueId and repository are required');
}

2. Consider using a validation library like Joi or Zod if you want more robust validation
3. Add TypeScript types if the project uses TypeScript

[sourcery-ai]

issue (complexity): Consider extracting common text content logic and flattening the code structure in the ADF to Markdown conversion.

The adfToMarkdown function can be simplified by extracting common patterns and reducing nesting. Here's a cleaner approach:

const getTextContent = (content) => {
  if (!content) return '';
  return content.map(item => item.text || '').join('');
};

const adfToMarkdown = (adf) => {
  if (!adf?.content) return '';

  return adf.content.map((block) => {
    const content = block.content || [];

    switch (block.type) {
      case 'paragraph':
        return getTextContent(content) + '\n\n';
      case 'heading':
        const level = block.attrs?.level || 1;
        return '#'.repeat(level) + ' ' + getTextContent(content) + '\n\n';
      case 'bulletList':
        return content.map(item => `- ${getTextContent(item.content)}`).join('\n') + '\n\n';
      case 'orderedList':
        return content.map((item, i) => `${i + 1}. ${getTextContent(item.content)}`).join('\n') + '\n\n';
      case 'codeBlock':
        return '```\n' + getTextContent(content) + '\n```\n\n';
      default:
        return '';
    }
  }).join('').trim();
};

This refactoring:

1. Extracts common text content mapping into getTextContent
2. Reduces nesting by flattening the structure
3. Maintains null safety while being more readable
4. Eliminates redundant optional chaining