Skip to content

improve the way AGENTS.md is generated - #1751

Merged
llrs-roche merged 1 commit into
mainfrom
fix-agents-rmd@main
Oct 20, 2026
Merged

llrs-roche merged 1 commit into
mainfrom
fix-agents-rmd@main

Conversation

@osenan

@osenan osenan commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

The way _AGENTS.Rmd parsed the optional section was not up to date, this PR is addressing that. I already generated the new AGENTS.md which is the same.

@osenan
osenan requested a review from a team October 14, 2026 15:55
@github-actions

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                          Stmts    Miss  Cover    Missing
------------------------------  -------  ------  -------  -----------------------------------------------------------------------------------------------------
R/after.R                            59      21  64.41%   42-52, 64, 69, 77-79, 81-89, 100, 104-105
R/checkmate.R                        24       0  100.00%
R/decorators-helpers.R               59       1  98.31%   113
R/dummy_functions.R                  61       2  96.72%   54, 56
R/include_css_js.R                   11       0  100.00%
R/init.R                            136       0  100.00%
R/module_bookmark_manager.R          99      54  45.45%   78-133
R/module_data_summary.R             177       8  95.48%   40, 50, 205, 236-240
R/module_filter_data.R               64       0  100.00%
R/module_filter_manager.R           210       7  96.67%   119-120, 316, 343, 355, 362-363
R/module_nested_tabs.R              411      42  89.78%   120, 178-183, 211, 315-330, 350-354, 409, 527-530, 534-537, 541-544
R/module_session_info.R              18       0  100.00%
R/module_snapshot_manager.R         276       9  96.74%   315-319, 386, 389-391
R/module_source_code.R               69       0  100.00%
R/module_teal_lockfile.R            131      53  59.54%   45-57, 60-62, 76, 86-88, 100-102, 110-119, 122, 124, 126-127, 142-146, 161-162, 177-186
R/module_teal_reporter.R            122       9  92.62%   60, 77-78, 81, 98, 128, 142, 144, 158
R/module_teal.R                     310      26  91.61%   137, 186, 242, 272-292, 322-323
R/module_transform_data.R           164      25  84.76%   77-81, 85-89, 106, 112, 119-120, 134-138, 188, 273-277
R/module_validate_error.R            73       0  100.00%
R/modules.R                         345      50  85.51%   170-174, 229-232, 332, 339, 353, 472, 570-576, 706-712, 725-733, 748-763, 796, 808-816
R/reporter_previewer_module.R        41      12  70.73%   41, 45, 68-85
R/teal_data_module-eval_code.R       23       0  100.00%
R/teal_data_module-within.R           7       0  100.00%
R/teal_data_module.R                 23       0  100.00%
R/teal_data_utils.R                  49       0  100.00%
R/teal_modifiers.R                   57       0  100.00%
R/teal_slices-store.R                29       0  100.00%
R/teal_slices.R                      48       2  95.83%   153-154
R/teal_transform_module.R            45       0  100.00%
R/TealAppDriver.R                   353     244  30.88%   68-73, 126, 134-163, 194-203, 213, 228-276, 301-564, 592-617, 672, 674-677, 696-710, 727-728, 739-756
R/utils.R                           290      48  83.45%   404-453, 541-550
R/validate_inputs.R                  60       0  100.00%
R/validations.R                      58       0  100.00%
R/zzz.R                              21       0  100.00%
TOTAL                              3923     613  84.37%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 4244e4f

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   32 suites   2m 48s ⏱️
471 tests 419 ✅ 52 💤 0 ❌
748 runs  692 ✅ 56 💤 0 ❌

Results for commit 4244e4f.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
module_session_info 💔 $18.33$ $+2.12$ $0$ $0$ $0$ $0$
module_teal 💚 $124.68$ $-11.77$ $0$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
module_session_info 💔 $18.31$ $+2.12$ srv_session_info_lockfile_creation_process_is_invoked_for_teal.lockfile.mode_enabled_and_snapshot_is_copied_to_teal_app.lock_and_removed_after_session_ended

Results for commit ea0e94b

♻️ This comment has been updated with latest results.

@wangben718 wangben718 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me to have this improvement!

@osenan

osenan commented Oct 20, 2026

Copy link
Copy Markdown
Contributor Author

The AGENTS.md was re-rendered, but the result was exactly the same.
insightsengineering/teal.widgets#368 This PR is related

@llrs-roche
llrs-roche merged commit 79e34d1 into main Oct 20, 2026
28 checks passed
@llrs-roche
llrs-roche deleted the fix-agents-rmd@main branch October 20, 2026 15:26
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants