Skip to content

Conversation

@varin-nair-factory
Copy link
Collaborator

@varin-nair-factory varin-nair-factory commented Jan 9, 2026

Summary

Apply consistent code formatting across the codebase using Prettier.

Changes

  • Standardize markdown list markers (asterisks to dashes)
  • Fix line length and indentation in source files
  • Format test files for consistency
  • Normalize trailing commas and whitespace

Testing

  • All 308 tests pass
  • Typecheck passes

Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@varin-nair-factory varin-nair-factory marked this pull request as draft January 9, 2026 19:33
@varin-nair-factory varin-nair-factory marked this pull request as ready for review January 9, 2026 19:33
Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@varin-nair-factory varin-nair-factory marked this pull request as draft January 9, 2026 19:34
@varin-nair-factory varin-nair-factory marked this pull request as ready for review January 9, 2026 19:34
Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@varin-nair-factory varin-nair-factory marked this pull request as draft January 9, 2026 19:40
@varin-nair-factory varin-nair-factory marked this pull request as ready for review January 9, 2026 19:41
Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@Factory-AI Factory-AI deleted a comment from factory-droid bot Jan 9, 2026
@varin-nair-factory varin-nair-factory marked this pull request as draft January 9, 2026 19:47
@varin-nair-factory varin-nair-factory marked this pull request as ready for review January 9, 2026 19:47
@factory-droid
Copy link
Contributor

factory-droid bot commented Jan 9, 2026

Droid finished @varin-nair-factory's task —— View job


"--output-format",
"stream-json",
"--skip-permissions-unsafe",
"--skip-permissions-unsafe",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P0] Fix string element missing indentation (syntax error)

This line is missing array indentation (it starts at column 1), which makes the test file invalid TS/JS and will cause parsing/CI failures.

beforeAll(async () => {
const module = (await import(
`../src/run-droid?mcp-test=${Math.random().toString(36).slice(2)}`,
`../src/run-droid?mcp-test=${Math.random().toString(36).slice(2)}`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P0] Fix invalid dynamic import syntax (missing trailing comma)

This multiline import( call is missing the trailing comma after the template string, which makes the file invalid syntax and will break the test suite.

Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Formatting changes look mostly fine, but the PR introduces a couple of syntax-breaking edits in base-action tests that will fail parsing/CI. Please fix the broken array indentation and the missing comma in the dynamic import, then rerun tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants