feat: improve global variables(time, file, dataset)

[fishwww-ww]

No description provided.

[gru-agent]

TestGru Assignment

Summary

Link	CommitId	Status	Reason
Detail	f052bb6	✅ Finished

History Assignment

Files

File	Pull Request
projects/app/src/components/core/app/formRender/utils.ts	❌ Failed (Something wrong with the environment: The test runner cannot resolve the import path '../../../../projects/app/src/components/core/app/formRender/utils' from the test file. This is an environmental/configuration issue, not a source code bug. The test code and source code are correct, but the test cannot run due to incorrect or missing file resolution in the test environment.)

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

[github-actions]

Preview mcp_server Image:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-pr:fatsgpt_mcp_server_be7722433f01edc2e7d51e4d9daf1bbb9055aaf1

[github-actions]

Preview sandbox Image:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-pr:fatsgpt_sandbox_be7722433f01edc2e7d51e4d9daf1bbb9055aaf1

[github-actions]

Preview fastgpt Image:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-pr:fatsgpt_be7722433f01edc2e7d51e4d9daf1bbb9055aaf1

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[cursor]

Bug: Unintended Debugging Artifacts Committed

Two debugging artifacts, a stray 1; statement and a console.log call, appear to have been accidentally committed and serve no functional purpose.

Additional Locations (1)

• projects/app/src/components/core/app/formRender/index.tsx#L290-L291

 

[cursor]

Bug: TimeInput Component Redundant Date Formatting

The TimeInput component unnecessarily formats startDate and endDate with formatTime2YMDHMS before creating a Date object. This redundant step can cause parsing issues or precision loss, potentially leading to invalid date values in the time range selector.

 

[cursor]

Bug: Date Parsing Error Causes Formatting Issues

The new Date(data.timePointDefault || '') expression can create an invalid Date object if data.timePointDefault is falsy. This invalid date, when passed to formatTime2YMDHMS, may cause unexpected results or runtime errors.

 

[cursor]

Bug: Redundant Date Formatting Causes Parsing Errors

Unnecessary and potentially error-prone date formatting: The code calls formatTime2YMDHMS(startDate) to format a date to string, then immediately creates a new Date from that string with new Date(formatTime2YMDHMS(startDate)). This double conversion could introduce parsing errors and is redundant. If startDate needs to be a Date object, it should be converted directly without the intermediate string formatting.

Additional Locations (1)

• projects/app/src/components/core/app/formRender/index.tsx#L371-L372

 

[cursor]

Bug: Stale State in Asynchronous Callback

The handleAddUrl function uses stale state in the onChange callback. It calls onChange([...urlFileList, trimmedUrl]) immediately after setUrlFileList((prev) => [...prev, trimmedUrl]), but since state updates are asynchronous, urlFileList still contains the old value. This causes the parent component to receive an incomplete file list. The fix should be to use the new array value: const newList = [...urlFileList, trimmedUrl]; setUrlFileList(newList); onChange(newList);